This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [RFC] embryo of type/main_type pretty_printing...


Thanks, Daniel and Tom, for taking the time to review the script!
Once checked in, please feel free to rework/enhance as you see fit.
I'll definitely get to it again, but I really want to look at my
DW_AT_GNAT_descriptive_type patch, which will make the gnat-stuff
thing much simpler. I'm pretty sure the patch is ready, but I want
to have another good read + testing before posting.

> Regardless of whether you decide to change this, I think you should
> check this in.  That will make it simpler for us to improve it.

I agree that checking it in would be useful.  I just don't know where,
yet. Should we start a "contrib" directory? Or just in the gdb/ directory
itself if we consider this as port of the GDB sources?

I had also had questions about whether we want to enable this script
by default or not. For now, obviously not, since we really have not
finished discussing the output format (not to mention testing). But
this affects, IMO, the location where we want to store the script.
I was thinking about having the "make" install that script in the
build directory besides the gdb exe so that it gets automatically
used.  Or perhaps we can use a different strategy of adding the source
path to one of the search paths that GDB uses... WDYT?

-- 
Joel


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]