[PATCH v2 0/1] Correct argument description for info connections/inferiors

[PATCH v2 0/1] Correct argument description for info connections/inferiors

[Christina Schimpe]

Hi all,

this is my v2 for the patch "gdb, doc: correct argument description for info
connections/inferiors", which addresses Eli's feedback.

V1 of this patch can be found here: 
https://sourceware.org/pipermail/gdb-patches/2023-March/198311.html

Regards,
Christina

Nils-Christian Kempke (1):
  gdb, doc: correct argument description for info connections/inferiors

 gdb/doc/gdb.texinfo | 19 +++++++++++++------
 1 file changed, 13 insertions(+), 6 deletions(-)

-- 
2.25.1

Intel Deutschland GmbH
Registered Address: Am Campeon 10, 85579 Neubiberg, Germany
Tel: +49 89 99 8853-0, www.intel.de <http://www.intel.de>
Managing Directors: Christin Eisenschmid, Sharon Heck, Tiffany Doon Silva  
Chairperson of the Supervisory Board: Nicole Lau
Registered Office: Munich
Commercial Register: Amtsgericht Muenchen HRB 186928

[PATCH v2 1/1] gdb, doc: correct argument description for info connections/inferiors

[Christina Schimpe]

From: Nils-Christian Kempke <nils-christian.kempke@intel.com>

It said for 'info inferiors' and 'info connections' that the argument
could be 'a space separated list of inferior numbers' which is correct
but incomplete.  In fact the arguments can be any space separated
combination of numbers and (ascending) ranges.

The beginning of the section now describes the ID list as a new keyword.

Co-Authored-By: Christina Schimpe <christina.schimpe@intel.com>
---
 gdb/doc/gdb.texinfo | 19 +++++++++++++------
 1 file changed, 13 insertions(+), 6 deletions(-)

diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo
index 92d99349dbf..3ec267a7825 100644
--- a/gdb/doc/gdb.texinfo
+++ b/gdb/doc/gdb.texinfo
@@ -3216,6 +3216,15 @@ embedded targets may have several inferiors running in different parts
 of a single address space.  Each inferior may in turn have multiple
 threads running in it.
 
+@cindex ID list
+The commands @code{info inferiors} and @code{info connections}, which will be
+introduced below, accept a space-separated @dfn{ID list} as their argument
+specifying one or more elements on which to operate.  A list element can be
+either a single non-negative number, like @samp{5}, or an ascending range of
+such numbers, like @samp{5-7}.  A list can consist of any combination of such
+elements, even duplicates or overlapping ranges are valid.  E.g.@:
+@samp{1 4-6 5 4-4} or @samp{1 2 4-7}.
+
 To find out what inferiors exist at any moment, use @w{@code{info
 inferiors}}:
 
@@ -3223,9 +3232,8 @@ inferiors}}:
 @kindex info inferiors [ @var{id}@dots{} ]
 @item info inferiors
 Print a list of all inferiors currently being managed by @value{GDBN}.
-By default all inferiors are printed, but the argument @var{id}@dots{}
--- a space separated list of inferior numbers -- can be used to limit
-the display to just the requested inferiors.
+By default all inferiors are printed, but the ID list @var{id}@dots{} can be
+used to limit the display to just the requested inferiors.
 
 @value{GDBN} displays for each inferior (in this order):
 
@@ -3284,9 +3292,8 @@ To find out what open target connections exist at any moment, use
 @kindex info connections [ @var{id}@dots{} ]
 @item info connections
 Print a list of all open target connections currently being managed by
-@value{GDBN}.  By default all connections are printed, but the
-argument @var{id}@dots{} -- a space separated list of connections
-numbers -- can be used to limit the display to just the requested
+@value{GDBN}.  By default all connections are printed, but the ID list
+@var{id}@dots{} can be used to limit the display to just the requested
 connections.
 
 @value{GDBN} displays for each connection (in this order):
-- 
2.25.1

Intel Deutschland GmbH
Registered Address: Am Campeon 10, 85579 Neubiberg, Germany
Tel: +49 89 99 8853-0, www.intel.de <http://www.intel.de>
Managing Directors: Christin Eisenschmid, Sharon Heck, Tiffany Doon Silva  
Chairperson of the Supervisory Board: Nicole Lau
Registered Office: Munich
Commercial Register: Amtsgericht Muenchen HRB 186928

Re: [PATCH v2 1/1] gdb, doc: correct argument description for info connections/inferiors

[Eli Zaretskii]

> From: Christina Schimpe <christina.schimpe@intel.com>
> Cc: eliz@gnu.org,
> 	Christina Schimpe <christina.schimpe@intel.com>
> Date: Thu, 30 Mar 2023 20:02:39 +0200
> 
> From: Nils-Christian Kempke <nils-christian.kempke@intel.com>
> 
> It said for 'info inferiors' and 'info connections' that the argument
> could be 'a space separated list of inferior numbers' which is correct
> but incomplete.  In fact the arguments can be any space separated
> combination of numbers and (ascending) ranges.
> 
> The beginning of the section now describes the ID list as a new keyword.
> 
> Co-Authored-By: Christina Schimpe <christina.schimpe@intel.com>
> ---
>  gdb/doc/gdb.texinfo | 19 +++++++++++++------
>  1 file changed, 13 insertions(+), 6 deletions(-)

Thanks, this is OK.

Approved-By: Eli Zaretskii <eliz@gnu.org>

RE: [PATCH v2 1/1] gdb, doc: correct argument description for info connections/inferiors

[Schimpe, Christina]

Thank you for the review, I pushed this.

Christina

> -----Original Message-----
> From: Eli Zaretskii <eliz@gnu.org>
> Sent: Thursday, March 30, 2023 8:13 PM
> To: Schimpe, Christina <christina.schimpe@intel.com>
> Cc: gdb-patches@sourceware.org
> Subject: Re: [PATCH v2 1/1] gdb, doc: correct argument description for info
> connections/inferiors
> 
> > From: Christina Schimpe <christina.schimpe@intel.com>
> > Cc: eliz@gnu.org,
> > 	Christina Schimpe <christina.schimpe@intel.com>
> > Date: Thu, 30 Mar 2023 20:02:39 +0200
> >
> > From: Nils-Christian Kempke <nils-christian.kempke@intel.com>
> >
> > It said for 'info inferiors' and 'info connections' that the argument
> > could be 'a space separated list of inferior numbers' which is correct
> > but incomplete.  In fact the arguments can be any space separated
> > combination of numbers and (ascending) ranges.
> >
> > The beginning of the section now describes the ID list as a new keyword.
> >
> > Co-Authored-By: Christina Schimpe <christina.schimpe@intel.com>
> > ---
> >  gdb/doc/gdb.texinfo | 19 +++++++++++++------
> >  1 file changed, 13 insertions(+), 6 deletions(-)
> 
> Thanks, this is OK.
> 
> Approved-By: Eli Zaretskii <eliz@gnu.org>
Intel Deutschland GmbH
Registered Address: Am Campeon 10, 85579 Neubiberg, Germany
Tel: +49 89 99 8853-0, www.intel.de <http://www.intel.de>
Managing Directors: Christin Eisenschmid, Sharon Heck, Tiffany Doon Silva  
Chairperson of the Supervisory Board: Nicole Lau
Registered Office: Munich
Commercial Register: Amtsgericht Muenchen HRB 186928