This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [RFA] -list-target-groups --available
- From: Eli Zaretskii <eliz at gnu dot org>
- To: Vladimir Prus <vladimir at codesourcery dot com>
- Cc: drow at false dot org, gdb-patches at sources dot redhat dot com
- Date: Mon, 01 Dec 2008 22:37:39 +0200
- Subject: Re: [RFA] -list-target-groups --available
- References: <200811142003.33600.vladimir@codesourcery.com> <ud4gkj3bc.fsf@gnu.org> <u3ahgj01j.fsf@gnu.org> <200812011111.21264.vladimir@codesourcery.com>
- Reply-to: Eli Zaretskii <eliz at gnu dot org>
> From: Vladimir Prus <vladimir@codesourcery.com>
> Date: Mon, 1 Dec 2008 11:11:20 +0300
> Cc: drow@false.org, gdb-patches@sources.redhat.com
>
> > . @appendix is an equivalent of @section, so its immediate sub-nodes
> > cannot be @section's. They should be @appendixsubsec instead.
>
> That lead to texinfo/tex errors. @appendixsection, however, appears to
> work fine both to info and pdf output.
Sorry, my bad. Yes, @appendixsection is the right command.
> > > +Each item is required to include a column with name of @samp{pid},
> > > +that identifies the process.
> >
> > "name of `pid'" does not sound right. Do you mean " the name of the
> > process identified by @var{pid}"?
>
> I mean that:
>
> - the name of the column should be 'pid'
> - the value of the column should identify the process
>
> I've clarified this and also addressed Dan's comments. Revised patch
> attached, OK?
Yes, but I have a better suggestion:
> +Each item should include a column with name of @samp{pid}.
Each item should include a column whose name is @samp{pid}.