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 / remote protocol] Remote shared library events


On Fri, May 11, 2007 at 11:02:50AM -0700, Kevin Buettner wrote:
> On Wed, 9 May 2007 16:16:27 -0400
> Daniel Jacobowitz <drow@false.org> wrote:
> 
> > What do you think, Kevin especially?  I'm not entirely thrilled with
> > the way this interacts with other solib_ops vectors, but I'm not
> > unhappy with it either.
> 
> What in particular were you not thrilled with?

I had to add two new methods, add_one_library and remove_one_library,
that don't make sense for any of the other things using solib_ops.
I thought for a bit about making this something completely different,
so that you could have both a solib_ops and target-reported libraries;
I think we'll want multiple sorts of libraries someday, but today
I couldn't make it work sensibly.

Also, the in_dynsym_code method is a bit hokey; calling in_plt_section
happens to be right for SymbianOS, but I have a feeling Windows will
want something more (at least, the relevant test is gdb1555.exp, and
it fails on Windows once I add testsuite support to run it there).
I suspect this should be split into a solib_ops specific bit and an
architecture specific bit, or maybe just an architecture specific bit.

Neither of these are big problems, really.

> I've looked over the solib-specific changes.  They look reasonable to
> me.  I didn't understand the infrun changes.  I too would like to see
> that put into a different patch.

Coming right up (well, probably Monday at this point).  Test case for
it, too.

-- 
Daniel Jacobowitz
CodeSourcery


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