Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Daniel Jacobowitz <drow@false.org>
To: Vladimir Prus <vladimir@codesourcery.com>
Cc: gdb-patches@sources.redhat.com
Subject: Re: [Patch] Fix for -var-update to use natural format to compare
Date: Tue, 22 Jan 2008 17:59:00 -0000	[thread overview]
Message-ID: <20080122175900.GA12246@caradoc.them.org> (raw)
In-Reply-To: <fn5abr$924$1@ger.gmane.org>

On Tue, Jan 22, 2008 at 08:49:53PM +0300, Vladimir Prus wrote:
> It's good to improve docs in this way. It should be noted that this
> might not be true in future, say when we add Python scripting,  or
> (maybe) when we'll add a way to specify floating point precision.

Let's keep the documentation focused on what's true today - no point
saying "this might not work someday" when there isn't an alternative
yet.

> > Also, if GDB requires this of frontends showing mutliple formats, it would
> > be nice to have -var-evaluate-expression take a format as an optional
> > parameter.  Then, the frontend would not need to use -var-set-format
> > to get a different formatted value, and could keep the variable object
> > in the natural format at all times.
> > 
> > I can provide a patch if this is a supported idea.
> 
> I personally think this is a good idea! It means that if we for example 
> need to quickly show some value in specific format, like hex value in tooltip,
> we don't need two format setting commands before and after. So, such
> a patch will be valuable independently of how -var-update operates.

I think this is a good idea too.

-- 
Daniel Jacobowitz
CodeSourcery


  reply	other threads:[~2008-01-22 17:59 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-01-22  3:25 Marc Khouzam
2008-01-22  3:53 ` Daniel Jacobowitz
2008-01-22 14:43   ` Marc Khouzam
2008-01-22 15:03     ` Daniel Jacobowitz
2008-01-22 15:12       ` Marc Khouzam
2008-01-22 15:35       ` Vladimir Prus
2008-01-22 16:38       ` Marc Khouzam
2008-01-22 17:50         ` Vladimir Prus
2008-01-22 17:59           ` Daniel Jacobowitz [this message]
2008-01-23  0:33     ` Nick Roberts
2008-01-23 14:41       ` Marc Khouzam

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=20080122175900.GA12246@caradoc.them.org \
    --to=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