public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
* [PATCH] gdb/doc: improve 'show print elements' description
@ 2021-10-06 11:31 Andrew Burgess
  2021-10-06 12:47 ` Tom de Vries
  2021-10-06 13:01 ` Eli Zaretskii
  0 siblings, 2 replies; 3+ messages in thread
From: Andrew Burgess @ 2021-10-06 11:31 UTC (permalink / raw)
  To: gdb-patches

The documentation for 'show print elements' contains the line:

  If the number is 0, then the printing is unlimited.

However, this line is now out of date as can be seen by this GDB
session:

  (gdb) set print elements 0
  (gdb) show print elements
  Limit on string chars or array elements to print is unlimited.

The value 0 does indeed mean unlimited, and this is described in the
'set print elements' section, however, for 'show print elements' the
user will never see the value 0, so lets just remove that bit from the
docs.
---
 gdb/doc/gdb.texinfo | 1 -
 1 file changed, 1 deletion(-)

diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo
index c156a1d6739..a6f207a41a7 100644
--- a/gdb/doc/gdb.texinfo
+++ b/gdb/doc/gdb.texinfo
@@ -11376,7 +11376,6 @@
 
 @item show print elements
 Display the number of elements of a large array that @value{GDBN} will print.
-If the number is 0, then the printing is unlimited.
 
 @anchor{set print frame-arguments}
 @item set print frame-arguments @var{value}
-- 
2.25.4


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] gdb/doc: improve 'show print elements' description
  2021-10-06 11:31 [PATCH] gdb/doc: improve 'show print elements' description Andrew Burgess
@ 2021-10-06 12:47 ` Tom de Vries
  2021-10-06 13:01 ` Eli Zaretskii
  1 sibling, 0 replies; 3+ messages in thread
From: Tom de Vries @ 2021-10-06 12:47 UTC (permalink / raw)
  To: Andrew Burgess, gdb-patches

On 10/6/21 1:31 PM, Andrew Burgess wrote:
> The documentation for 'show print elements' contains the line:
> 
>   If the number is 0, then the printing is unlimited.
> 
> However, this line is now out of date as can be seen by this GDB
> session:
> 
>   (gdb) set print elements 0
>   (gdb) show print elements
>   Limit on string chars or array elements to print is unlimited.
> 
> The value 0 does indeed mean unlimited, and this is described in the
> 'set print elements' section, however, for 'show print elements' the
> user will never see the value 0, so lets just remove that bit from the
> docs.

LGTM..

Thanks,
- Tom

> ---
>  gdb/doc/gdb.texinfo | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo
> index c156a1d6739..a6f207a41a7 100644
> --- a/gdb/doc/gdb.texinfo
> +++ b/gdb/doc/gdb.texinfo
> @@ -11376,7 +11376,6 @@
>  
>  @item show print elements
>  Display the number of elements of a large array that @value{GDBN} will print.
> -If the number is 0, then the printing is unlimited.
>  
>  @anchor{set print frame-arguments}
>  @item set print frame-arguments @var{value}
> 

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] gdb/doc: improve 'show print elements' description
  2021-10-06 11:31 [PATCH] gdb/doc: improve 'show print elements' description Andrew Burgess
  2021-10-06 12:47 ` Tom de Vries
@ 2021-10-06 13:01 ` Eli Zaretskii
  1 sibling, 0 replies; 3+ messages in thread
From: Eli Zaretskii @ 2021-10-06 13:01 UTC (permalink / raw)
  To: Andrew Burgess; +Cc: gdb-patches

> From: Andrew Burgess <andrew.burgess@embecosm.com>
> Date: Wed,  6 Oct 2021 12:31:59 +0100
> 
> diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo
> index c156a1d6739..a6f207a41a7 100644
> --- a/gdb/doc/gdb.texinfo
> +++ b/gdb/doc/gdb.texinfo
> @@ -11376,7 +11376,6 @@
>  
>  @item show print elements
>  Display the number of elements of a large array that @value{GDBN} will print.
> -If the number is 0, then the printing is unlimited.
>  
>  @anchor{set print frame-arguments}
>  @item set print frame-arguments @var{value}

Thanks, this is OK.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-10-06 13:01 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-10-06 11:31 [PATCH] gdb/doc: improve 'show print elements' description Andrew Burgess
2021-10-06 12:47 ` Tom de Vries
2021-10-06 13:01 ` Eli Zaretskii

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).