From: Daniel Jacobowitz <drow@false.org>
To: Joel Brobecker <brobecker@adacore.com>
Cc: gdb-patches@sources.redhat.com
Subject: Re: [RFC] Replace deprecated_target_new_objfile_hook by observer?
Date: Tue, 17 Oct 2006 23:35:00 -0000 [thread overview]
Message-ID: <20061017233524.GA18064@nevyn.them.org> (raw)
In-Reply-To: <20061017233217.GS1903@adacore.com>
On Tue, Oct 17, 2006 at 07:32:17PM -0400, Joel Brobecker wrote:
> I propose to replace this with an observer. Would that be OK?
There Be Dragons! Note that there is a case where the remote objfile
hook does _not_ call the next thing on the chain. This is somewhat
deliberate, in that it's what prevents thread-db from being enabled
when talking to gdbserver, and somewhat accidental, in that I'm sure it
wasn't meant to work this way. But we'll need to do some rearranging
in order to keep the current state of affairs working.
> Assuming that it is, there are several platforms that use that
> mechanism. It's going to be hard to test all of them. But the changes
> themselves should be pretty mechanical. So what I can propose is
> to make all the necessary changes to replace that hook model with
> an approach using observers and test on x86-linux. And then rely
> on the mechanical aspect of the change together with the review of
> another pair of eyes. Would that be OK?
Isn't there already an adequate observer? Well, not a single one I
guess, you'd have to survey the uses and see what they really wanted.
But solib_loaded goes a long way.
--
Daniel Jacobowitz
CodeSourcery
next prev parent reply other threads:[~2006-10-17 23:35 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-10-17 23:32 Joel Brobecker
2006-10-17 23:35 ` Daniel Jacobowitz [this message]
2006-10-18 0:20 ` Joel Brobecker
2006-10-18 1:38 ` Daniel Jacobowitz
[not found] ` <535BF17A-E776-4DE4-979B-7E6FBA505E31@apple.com>
2006-10-18 17:11 ` Daniel Jacobowitz
2006-10-18 16:29 ` Ulrich Weigand
2006-10-18 16:43 ` Daniel Jacobowitz
2006-10-19 14:30 ` Joel Brobecker
2006-10-20 0:41 ` [PATCH] " Ulrich Weigand
2006-10-20 0:46 ` Daniel Jacobowitz
2006-10-20 1:09 ` Ulrich Weigand
2007-03-28 18:34 ` Ulrich Weigand
2007-03-28 18:42 ` Joel Brobecker
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20061017233524.GA18064@nevyn.them.org \
--to=drow@false.org \
--cc=brobecker@adacore.com \
--cc=gdb-patches@sources.redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox