Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Nick Roberts <nickrob@snap.net.nz>
To: Daniel Jacobowitz <drow@false.org>
Cc: Vladimir Prus <vladimir@codesourcery.com>,
		Eli Zaretskii <eliz@gnu.org>,
	gdb-patches@sources.redhat.com
Subject: Re: Ping: frozen variable objects
Date: Sat, 14 Apr 2007 23:18:00 -0000	[thread overview]
Message-ID: <17953.23395.177793.84085@farnswood.snap.net.nz> (raw)
In-Reply-To: <20070414161353.GA24574@caradoc.them.org>

 > >  The range of values for fields with specified values, e.g.,
 > > -@code{in_scope} (@pxref{-var-update}) may be extended.
 > > +@code{in_scope} (@pxref{-var-update-fields}) may be extended.
 > 
 > Doesn't that read a little odd to you?  I'd think '-var-update fields'
 > at least.

Also the name that is displayed shouldn't change, so pxref needs to be
called with two arguments:


@code{in_scope} (@pxref{-var-update fields, -var-update}) may be extended.

and

@anchor{-var-update fields}
The field in_scope may take three values:

Here are some other observations about the documentation:

frozenness (not a common word) has two ns.

In the description for -var-set-frozen:

  parameter should be either `1' to make the variable frozen or `0' to
  make it unfozen.  If a variable object is frozen, then neither itself,
          ^^^^^^^

In the description for -var-update:

  If `*' is used as the variable object names, all existing variable objects
  are updated.

Since frozen variables aren't updated implicitly, I would move some
of the description of them from -var-set-frozen to here.

  @c The format of field's content e.g type prefix, may change so parse it
   @c   at your own risk.  Yes, in general?
  @@ -19536,7 +19536,10 @@ that corresponds to a composite type, su
   a number of child variable objects, for example corresponding to each
   element of a structure.  A child variable object can itself have 
   children, recursively.  Recursion ends when we reach 
  -leaf variable objects, which always have built-in types.
  +leaf variable objects, which always have built-in types.  Child variable
  +objects are created only by explicit request, so if a frontend 
  +is not interested in the children of a particular variable object, no
  +child will be created.
 
   For a leaf variable object it is possible to obtain its value as a
   string, or set the value from a string.  String value can be also
  @@ -19548,7 +19551,19 @@ A frontend does not need to read the val
   the program stops.  Instead, MI provides an update command that lists all
   variable objects whose values has changed since the last update
   operation.  This considerably reduces the amount of data that must
  -be transferred to the frontend.
  +be transferred to the frontend.  As noted above, children variable
  +objects are created on demand, and only leaf variable objects have a
  +real value.  As result, gdb will read target memory only for leaf
  +variables that frontend has created.

Why not just:

  @c The format of field's content e.g type prefix, may change so parse it
   @c   at your own risk.  Yes, in general?
  @@ -19536,7 +19536,10 @@ that corresponds to a composite type, su
   a number of child variable objects, for example corresponding to each
   element of a structure.  A child variable object can itself have 
   children, recursively.  Recursion ends when we reach 
  -leaf variable objects, which always have built-in types.
  +leaf variable objects, which always have built-in types.  Child variable
  +objects are created only by explicit request, so if a frontend 
  +is not interested in the children of a particular variable object, no
  +child will be created.   As result, gdb will read target memory only for
  +leaf variables that frontend has created.


-- 
Nick                                           http://www.inet.net.nz/~nickrob


  reply	other threads:[~2007-04-14 22:55 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-03-25  9:52 Vladimir Prus
2007-03-25 20:17 ` Eli Zaretskii
2007-04-10 19:03 ` Daniel Jacobowitz
2007-04-11 14:44   ` Vladimir Prus
2007-04-11 14:52     ` Daniel Jacobowitz
2007-04-11 18:38       ` Eli Zaretskii
2007-04-13  9:53     ` Eli Zaretskii
2007-04-14 16:14       ` Vladimir Prus
2007-04-14 16:19         ` Daniel Jacobowitz
2007-04-14 23:18           ` Nick Roberts [this message]
2007-04-15 10:32             ` Vladimir Prus
2007-04-15 10:35             ` Vladimir Prus
2007-04-15 11:46               ` Nick Roberts
2007-04-15 11:57                 ` Vladimir Prus
2007-04-15 20:09                   ` Nick Roberts
2007-04-15 20:16                   ` Eli Zaretskii
2007-04-15 20:13               ` Eli Zaretskii
2007-04-14 20:32         ` Eli Zaretskii
2007-04-15 11:42           ` Vladimir Prus
2007-04-15 12:03             ` Nick Roberts
2007-04-16  2:40               ` Eli Zaretskii
2007-04-18  5:38                 ` Vladimir Prus
2007-04-18 10:37                   ` Nick Roberts
2007-04-18 22:23                     ` Eli Zaretskii
2007-04-19  4:41                   ` Eli Zaretskii
2007-04-15 20:14             ` Eli Zaretskii
2007-04-15  8:04         ` Nick Roberts
2007-04-18 10:22         ` [patch] fix insight (was: Re: Ping: frozen variable objects) Brian Dessent
2007-04-18 12:51           ` Maciej W. Rozycki
2007-04-18 13:01             ` Brian Dessent
2007-04-18  5:38   ` Ping: frozen variable objects Michael Snyder
2007-04-18  6:57     ` Vladimir Prus
2007-04-18 22:21       ` Michael Snyder

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=17953.23395.177793.84085@farnswood.snap.net.nz \
    --to=nickrob@snap.net.nz \
    --cc=drow@false.org \
    --cc=eliz@gnu.org \
    --cc=gdb-patches@sources.redhat.com \
    --cc=vladimir@codesourcery.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox