public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
* Metadata style?
@ 2022-03-28 18:46 Eli Zaretskii
  2022-04-07 11:14 ` Eli Zaretskii
  2022-04-07 18:44 ` Tom Tromey
  0 siblings, 2 replies; 8+ messages in thread
From: Eli Zaretskii @ 2022-03-28 18:46 UTC (permalink / raw)
  To: Tom Tromey; +Cc: gdb-patches

The command "show style" includes this part:

  style metadata background:  The "metadata" style background color is: none
  style metadata foreground:  The "metadata" style foreground color is: none
  style metadata intensity:  The "metadata" style display intensity is: dim

But there's no explanation in the manual what is the "metadata"
object.  What is it, and how can I display it in GDB to see the style
in action?

Thanks.

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

* Re: Metadata style?
  2022-03-28 18:46 Metadata style? Eli Zaretskii
@ 2022-04-07 11:14 ` Eli Zaretskii
  2022-04-07 18:44 ` Tom Tromey
  1 sibling, 0 replies; 8+ messages in thread
From: Eli Zaretskii @ 2022-04-07 11:14 UTC (permalink / raw)
  To: tromey; +Cc: gdb-patches

Ping!

> Date: Mon, 28 Mar 2022 21:46:03 +0300
> From: Eli Zaretskii via Gdb-patches <gdb-patches@sourceware.org>
> Cc: gdb-patches@sourceware.org
> 
> The command "show style" includes this part:
> 
>   style metadata background:  The "metadata" style background color is: none
>   style metadata foreground:  The "metadata" style foreground color is: none
>   style metadata intensity:  The "metadata" style display intensity is: dim
> 
> But there's no explanation in the manual what is the "metadata"
> object.  What is it, and how can I display it in GDB to see the style
> in action?
> 
> Thanks.
> 

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

* Re: Metadata style?
  2022-03-28 18:46 Metadata style? Eli Zaretskii
  2022-04-07 11:14 ` Eli Zaretskii
@ 2022-04-07 18:44 ` Tom Tromey
  2022-04-10  8:55   ` Eli Zaretskii
  1 sibling, 1 reply; 8+ messages in thread
From: Tom Tromey @ 2022-04-07 18:44 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: Tom Tromey, gdb-patches

>>>>> "Eli" == Eli Zaretskii <eliz@gnu.org> writes:

Eli> The command "show style" includes this part:
Eli>   style metadata background:  The "metadata" style background color is: none
Eli>   style metadata foreground:  The "metadata" style foreground color is: none
Eli>   style metadata intensity:  The "metadata" style display intensity is: dim

Eli> But there's no explanation in the manual what is the "metadata"
Eli> object.  What is it, and how can I display it in GDB to see the style
Eli> in action?

First, sorry about this, I completely slipped on writing documentation
for this.

There's a test for this style:

	gdb_test_no_output "set print repeat 3"
	gdb_test "print {0,0,0,0,0,0,0,0}" \
	    " = \\{0 [limited_style {<repeats.*8.*times>} metadata]\\}"

It's used in a number of places where gdb is attempting to display some
kind of data from the inferior but instead prints some
internally-generated text, e.g.:

  fprintf_styled (stream, metadata_style.style (), _("<unavailable>"));

I'm not completely sure of the best way to describe this.

Tom

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

* Re: Metadata style?
  2022-04-07 18:44 ` Tom Tromey
@ 2022-04-10  8:55   ` Eli Zaretskii
  2022-04-26 12:02     ` Eli Zaretskii
  2022-05-16 14:11     ` Tom Tromey
  0 siblings, 2 replies; 8+ messages in thread
From: Eli Zaretskii @ 2022-04-10  8:55 UTC (permalink / raw)
  To: Tom Tromey; +Cc: gdb-patches

> From: Tom Tromey <tromey@adacore.com>
> Cc: Tom Tromey <tromey@adacore.com>,  gdb-patches@sourceware.org
> Date: Thu, 07 Apr 2022 12:44:42 -0600
> 
> Eli> But there's no explanation in the manual what is the "metadata"
> Eli> object.  What is it, and how can I display it in GDB to see the style
> Eli> in action?
> 
> First, sorry about this, I completely slipped on writing documentation
> for this.
> 
> There's a test for this style:
> 
> 	gdb_test_no_output "set print repeat 3"
> 	gdb_test "print {0,0,0,0,0,0,0,0}" \
> 	    " = \\{0 [limited_style {<repeats.*8.*times>} metadata]\\}"
> 
> It's used in a number of places where gdb is attempting to display some
> kind of data from the inferior but instead prints some
> internally-generated text, e.g.:
> 
>   fprintf_styled (stream, metadata_style.style (), _("<unavailable>"));
> 
> I'm not completely sure of the best way to describe this.

Thanks.  How about the following addition to the manual?

--- gdb/doc/gdb.texinfo~0	2022-03-20 06:59:56.000000000 +0200
+++ gdb/doc/gdb.texinfo	2022-04-10 11:53:37.405750000 +0300
@@ -26283,6 +26283,16 @@
 the command @command{apropos -v REGEXP} uses the highlight style to
 mark the documentation parts matching @var{regexp}.
 
+@item metadata
+Control the styling of data annotations added by @value{GDBN} to data
+it displays.  By default, this style's intensity is dim.  Metadata
+annotations include the @samp{repeats @var{N} times} annotation for
+suppressed display of repeated array elements (@pxref{Print Strings}),
+@samp{<unavailable>} and @w{@samp{<error @var{descr}>}} annotations
+for errors and @samp{<optimized-out>} annotations for optimized-out
+values in displaying stack frame information in backtraces
+(@pxref{Backtrace}), etc.
+
 @item tui-border
 Control the styling of the TUI border.  Note that, unlike other
 styling options, only the color of the border can be controlled via

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

* Re: Metadata style?
  2022-04-10  8:55   ` Eli Zaretskii
@ 2022-04-26 12:02     ` Eli Zaretskii
  2022-05-04 12:40       ` Eli Zaretskii
  2022-05-16 14:11     ` Tom Tromey
  1 sibling, 1 reply; 8+ messages in thread
From: Eli Zaretskii @ 2022-04-26 12:02 UTC (permalink / raw)
  To: tromey; +Cc: gdb-patches

Ping!

> Date: Sun, 10 Apr 2022 11:55:48 +0300
> From: Eli Zaretskii via Gdb-patches <gdb-patches@sourceware.org>
> Cc: gdb-patches@sourceware.org
> 
> > From: Tom Tromey <tromey@adacore.com>
> > Cc: Tom Tromey <tromey@adacore.com>,  gdb-patches@sourceware.org
> > Date: Thu, 07 Apr 2022 12:44:42 -0600
> > 
> > Eli> But there's no explanation in the manual what is the "metadata"
> > Eli> object.  What is it, and how can I display it in GDB to see the style
> > Eli> in action?
> > 
> > First, sorry about this, I completely slipped on writing documentation
> > for this.
> > 
> > There's a test for this style:
> > 
> > 	gdb_test_no_output "set print repeat 3"
> > 	gdb_test "print {0,0,0,0,0,0,0,0}" \
> > 	    " = \\{0 [limited_style {<repeats.*8.*times>} metadata]\\}"
> > 
> > It's used in a number of places where gdb is attempting to display some
> > kind of data from the inferior but instead prints some
> > internally-generated text, e.g.:
> > 
> >   fprintf_styled (stream, metadata_style.style (), _("<unavailable>"));
> > 
> > I'm not completely sure of the best way to describe this.
> 
> Thanks.  How about the following addition to the manual?
> 
> --- gdb/doc/gdb.texinfo~0	2022-03-20 06:59:56.000000000 +0200
> +++ gdb/doc/gdb.texinfo	2022-04-10 11:53:37.405750000 +0300
> @@ -26283,6 +26283,16 @@
>  the command @command{apropos -v REGEXP} uses the highlight style to
>  mark the documentation parts matching @var{regexp}.
>  
> +@item metadata
> +Control the styling of data annotations added by @value{GDBN} to data
> +it displays.  By default, this style's intensity is dim.  Metadata
> +annotations include the @samp{repeats @var{N} times} annotation for
> +suppressed display of repeated array elements (@pxref{Print Strings}),
> +@samp{<unavailable>} and @w{@samp{<error @var{descr}>}} annotations
> +for errors and @samp{<optimized-out>} annotations for optimized-out
> +values in displaying stack frame information in backtraces
> +(@pxref{Backtrace}), etc.
> +
>  @item tui-border
>  Control the styling of the TUI border.  Note that, unlike other
>  styling options, only the color of the border can be controlled via
> 

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

* Re: Metadata style?
  2022-04-26 12:02     ` Eli Zaretskii
@ 2022-05-04 12:40       ` Eli Zaretskii
  0 siblings, 0 replies; 8+ messages in thread
From: Eli Zaretskii @ 2022-05-04 12:40 UTC (permalink / raw)
  To: tromey; +Cc: gdb-patches

Ping!  Ping!

> Date: Tue, 26 Apr 2022 15:02:46 +0300
> From: Eli Zaretskii via Gdb-patches <gdb-patches@sourceware.org>
> Cc: gdb-patches@sourceware.org
> 
> Ping!
> 
> > Date: Sun, 10 Apr 2022 11:55:48 +0300
> > From: Eli Zaretskii via Gdb-patches <gdb-patches@sourceware.org>
> > Cc: gdb-patches@sourceware.org
> > 
> > > From: Tom Tromey <tromey@adacore.com>
> > > Cc: Tom Tromey <tromey@adacore.com>,  gdb-patches@sourceware.org
> > > Date: Thu, 07 Apr 2022 12:44:42 -0600
> > > 
> > > Eli> But there's no explanation in the manual what is the "metadata"
> > > Eli> object.  What is it, and how can I display it in GDB to see the style
> > > Eli> in action?
> > > 
> > > First, sorry about this, I completely slipped on writing documentation
> > > for this.
> > > 
> > > There's a test for this style:
> > > 
> > > 	gdb_test_no_output "set print repeat 3"
> > > 	gdb_test "print {0,0,0,0,0,0,0,0}" \
> > > 	    " = \\{0 [limited_style {<repeats.*8.*times>} metadata]\\}"
> > > 
> > > It's used in a number of places where gdb is attempting to display some
> > > kind of data from the inferior but instead prints some
> > > internally-generated text, e.g.:
> > > 
> > >   fprintf_styled (stream, metadata_style.style (), _("<unavailable>"));
> > > 
> > > I'm not completely sure of the best way to describe this.
> > 
> > Thanks.  How about the following addition to the manual?
> > 
> > --- gdb/doc/gdb.texinfo~0	2022-03-20 06:59:56.000000000 +0200
> > +++ gdb/doc/gdb.texinfo	2022-04-10 11:53:37.405750000 +0300
> > @@ -26283,6 +26283,16 @@
> >  the command @command{apropos -v REGEXP} uses the highlight style to
> >  mark the documentation parts matching @var{regexp}.
> >  
> > +@item metadata
> > +Control the styling of data annotations added by @value{GDBN} to data
> > +it displays.  By default, this style's intensity is dim.  Metadata
> > +annotations include the @samp{repeats @var{N} times} annotation for
> > +suppressed display of repeated array elements (@pxref{Print Strings}),
> > +@samp{<unavailable>} and @w{@samp{<error @var{descr}>}} annotations
> > +for errors and @samp{<optimized-out>} annotations for optimized-out
> > +values in displaying stack frame information in backtraces
> > +(@pxref{Backtrace}), etc.
> > +
> >  @item tui-border
> >  Control the styling of the TUI border.  Note that, unlike other
> >  styling options, only the color of the border can be controlled via
> > 
> 

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

* Re: Metadata style?
  2022-04-10  8:55   ` Eli Zaretskii
  2022-04-26 12:02     ` Eli Zaretskii
@ 2022-05-16 14:11     ` Tom Tromey
  2022-05-16 16:06       ` Eli Zaretskii
  1 sibling, 1 reply; 8+ messages in thread
From: Tom Tromey @ 2022-05-16 14:11 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: Tom Tromey, gdb-patches

>>>>> "Eli" == Eli Zaretskii <eliz@gnu.org> writes:

Eli> +@item metadata
Eli> +Control the styling of data annotations added by @value{GDBN} to data
Eli> +it displays.  By default, this style's intensity is dim.  Metadata
Eli> +annotations include the @samp{repeats @var{N} times} annotation for
Eli> +suppressed display of repeated array elements (@pxref{Print Strings}),
Eli> +@samp{<unavailable>} and @w{@samp{<error @var{descr}>}} annotations
Eli> +for errors and @samp{<optimized-out>} annotations for optimized-out
Eli> +values in displaying stack frame information in backtraces
Eli> +(@pxref{Backtrace}), etc.

Thank you for writing this.  It looks good to me.

Tom

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

* Re: Metadata style?
  2022-05-16 14:11     ` Tom Tromey
@ 2022-05-16 16:06       ` Eli Zaretskii
  0 siblings, 0 replies; 8+ messages in thread
From: Eli Zaretskii @ 2022-05-16 16:06 UTC (permalink / raw)
  To: Tom Tromey; +Cc: gdb-patches

> From: Tom Tromey <tromey@adacore.com>
> Cc: Tom Tromey <tromey@adacore.com>,  gdb-patches@sourceware.org
> Date: Mon, 16 May 2022 08:11:12 -0600
> 
> >>>>> "Eli" == Eli Zaretskii <eliz@gnu.org> writes:
> 
> Eli> +@item metadata
> Eli> +Control the styling of data annotations added by @value{GDBN} to data
> Eli> +it displays.  By default, this style's intensity is dim.  Metadata
> Eli> +annotations include the @samp{repeats @var{N} times} annotation for
> Eli> +suppressed display of repeated array elements (@pxref{Print Strings}),
> Eli> +@samp{<unavailable>} and @w{@samp{<error @var{descr}>}} annotations
> Eli> +for errors and @samp{<optimized-out>} annotations for optimized-out
> Eli> +values in displaying stack frame information in backtraces
> Eli> +(@pxref{Backtrace}), etc.
> 
> Thank you for writing this.  It looks good to me.

Thanks for the review.  Installed on the master branch.

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

end of thread, other threads:[~2022-05-16 16:06 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-03-28 18:46 Metadata style? Eli Zaretskii
2022-04-07 11:14 ` Eli Zaretskii
2022-04-07 18:44 ` Tom Tromey
2022-04-10  8:55   ` Eli Zaretskii
2022-04-26 12:02     ` Eli Zaretskii
2022-05-04 12:40       ` Eli Zaretskii
2022-05-16 14:11     ` Tom Tromey
2022-05-16 16:06       ` 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).