This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: Move the multi-forks support to the generic multi-inferiors support.
- From: Pedro Alves <pedro at codesourcery dot com>
- To: Tom Tromey <tromey at redhat dot com>
- Cc: gdb-patches at sourceware dot org
- Date: Tue, 9 Jun 2009 00:17:00 +0100
- Subject: Re: Move the multi-forks support to the generic multi-inferiors support.
- References: <200905310013.38916.pedro@codesourcery.com> <200906061707.21594.pedro@codesourcery.com> <m3skiamjxj.fsf@fleche.redhat.com>
On Monday 08 June 2009 23:37:12, Tom Tromey wrote:
> Pedro> (I was a bit surprised to (re-)find that MI doesn't use
> Pedro> print_inferior)
>
> Naughty. Though at a quick glance I didn't see where MI does any
> printing of the inferiors...?
It's in the -list-thread-groups command (mi_cmd_list_thread_groups/print_one_inferior).
> In this case I think the patch is ok anyway -- an error in "info
> threads" does not, IMO, imply that we should propagate the mistake to
> other commands. Let me know what you think of this, I won't install
> the patch if you feel differently.
No, I agree with you.
My only concern is that the PID column may end up being
the wrong name for the target-id, due to targets that don't have a real
notion of PID. I was planning of making that column's value print
something like target_pid_to_str (pid_to_ptid (PID)), just like the
corresponding column in info threads. I was hoping that when
we'd get to name the columns in "info threads" we'd come up with
a nice name for it. :-)
--
Pedro Alves