Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Nick Roberts <nickrob@snap.net.nz>
To: Vladimir Prus <vladimir@codesourcery.com>
Cc: Daniel Jacobowitz <drow@false.org>, 	gdb-patches@sources.redhat.com
Subject: Re: MI: -var-list-children --simple-values
Date: Wed, 15 Nov 2006 20:29:00 -0000	[thread overview]
Message-ID: <17755.30642.667665.206754@kahikatea.snap.net.nz> (raw)
In-Reply-To: <200611151929.29567.vladimir@codesourcery.com>

 > > I didn't invent --simple-values though, I just added it to -var-update.
 > > Nick added it to -stack-list-locals on 20-Jan-2004.  Ah here:
 > >   http://sourceware.org/ml/gdb/2003-11/msg00055.html

Actually I think you added it to -var-update (where I think it makes
no difference as all values here are simple) and -var-list-children.  I think
this all started because when I suggested "--all-values" for -var-update
and -var-list-children Eli found the names unfortunate and suggested others.
I think the final solution was an effort to keep everyone happy.

 > This is in fact completely different change. Here's 
 > what "-stack-list-locals --all-values" produces:
 > 
 >       ^done,locals=[{name="the_struct",value="{a_fr = 120, b = 333, inner_fr =
 >            {i = 0, j = 5, k = 5}, c = 3, pad = {0 <repeats 100 times>}, d =
 >             10, pad2 = {0 <repeats 100 times>}, e = 10}"}]
 > 
 > So, the difference between --all-values and --simple-values is dramatic. In 
 > case of --var-list-children you don't get full structure printed. You won't 
 > even have the structure *read* (with my laziness patch, that is).

Yes, I describe the purpose in the manual:

  ...; and if it is 2 or `--simple-values', print the name, type and value for
  simple data types and the name and type for arrays, structures and
  unions.  In this last case, a frontend can immediately display the
  value of simple data types and create variable objects for other data
  types when the the user wishes to explore their values in more detail.

ISTR this is what Totalview does (not that it matters but it might help some
understand its purpose).

 > So, while --simple-values might be reasonable for -stack-list-locals, it's
 > not all that needed for -var-list-children.

I can't see where it's needed, but its more trouble to remove than to fix.

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


  reply	other threads:[~2006-11-15 20:29 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <200611151250.11654.vladimir@codesourcery.com>
2006-11-15 11:21 ` Nick Roberts
2006-11-15 11:50   ` Vladimir Prus
2006-11-15 14:44     ` Daniel Jacobowitz
2006-11-15 14:55       ` Vladimir Prus
2006-11-15 15:09         ` Daniel Jacobowitz
2006-11-15 16:29           ` Vladimir Prus
2006-11-15 20:29             ` Nick Roberts [this message]
2006-11-29  9:35               ` Vladimir Prus
2006-11-29  9:58                 ` Nick Roberts
2006-11-16 15:57             ` Daniel Jacobowitz
2006-11-15 19:55       ` Nick Roberts

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=17755.30642.667665.206754@kahikatea.snap.net.nz \
    --to=nickrob@snap.net.nz \
    --cc=drow@false.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