Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
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


  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