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
next prev parent 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