This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] Another annotation for threads
- From: Daniel Jacobowitz <drow at false dot org>
- To: Nick Roberts <nickrob at snap dot net dot nz>
- Cc: gdb-patches at sources dot redhat dot com
- Date: Thu, 5 Jun 2008 15:30:17 -0400
- Subject: Re: [PATCH] Another annotation for threads
- References: <18483.36546.101715.670386@kahikatea.snap.net.nz>
On Wed, May 21, 2008 at 02:53:54PM +1200, Nick Roberts wrote:
>
> This is a patch for another annotation, this time for thread changes. It
> follows an earlier patch (not committed) for notifying thread changes in MI
> using observers. In that patch thread changes were notified in two places:
> normal_stop in infrun.c and do_captured_thread_select in thread.c. Here
> normal_stop is used but the second call has been moved to thread_command so
> that temporary thread changes in commands like "info thread", "-var-update" etc
> aren't reported.
>
> In the doc patch, I've used @r{, annotation} with findex but I don't know what
> all this does as annotate.texinfo doesn't appear to have a function index.
>
> If this patch is OK, I will submit a similar one for MI using an observer.
It's quite likely that I've gotten turned around in all the discussion
while I was away, and you've already answered this. But here's my
question anyway: if there's going to be an observer to do this in MI,
why shouldn't annotate.c use the same observer to call
annotate_thread_changed?
--
Daniel Jacobowitz
CodeSourcery