Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Andrew STUBBS <andrew.stubbs@st.com>
To: Daniel Jacobowitz <drow@false.org>
Cc: Jim Blandy <jimb@red-bean.com>, Eli Zaretskii <eliz@gnu.org>,
	gdb-patches@sources.redhat.com
Subject: Re: [PATCH] keep-variable command
Date: Thu, 17 Nov 2005 15:07:00 -0000	[thread overview]
Message-ID: <437C7D3D.1090905@st.com> (raw)
In-Reply-To: <20051117035532.GD3057@nevyn.them.org>

Daniel Jacobowitz wrote:
> For one thing, why is this a command at all?  If we support keeping
> convenience variables across symbol file reloads, then why not do it by
> default for all convenience variables?  I doubt having them disappear
> has ever been considered a feature.

If it doesn't find a type for the variable it gives a message and sets 
the value to void. This would be irritating with a large number of 
variables you no longer care about. Otherwise I would agree with you.

Perhaps instead of the message the value could somehow be shown as 
'<type lost>', in which case it wouldn't matter so much. I was 
considering the possibility of keeping the value around on the off 
chance the type has come back the next time the variable is accessed.

> Alternatively, we could add code to duplicate the types recursively
> onto a convenience variable obstack.  GDB doesn't have much of a notion
> of "type compatibility".  It might not work 100% right for things like
> C++ operator overloading, but for that we'd need to do type merging
> anyway.

I have no idea what you just said, but if it avoids type fixup problems 
then that would be a good thing all round. Is it worth whatever effort 
it would take though? Is it instead of textual type remembering or 
instead of replacing the type at all?

Andrew


  reply	other threads:[~2005-11-17 12:56 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-11-16 15:54 Andrew STUBBS
2005-11-16 20:26 ` Eli Zaretskii
2005-11-17  1:07   ` Jim Blandy
2005-11-17  9:56     ` Daniel Jacobowitz
2005-11-17 15:07       ` Andrew STUBBS [this message]
2005-11-17 12:42     ` Andrew STUBBS
2005-11-17 12:26   ` Andrew STUBBS
2005-11-17 19:33     ` Eli Zaretskii
2005-11-17 19:36       ` Daniel Jacobowitz

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=437C7D3D.1090905@st.com \
    --to=andrew.stubbs@st.com \
    --cc=drow@false.org \
    --cc=eliz@gnu.org \
    --cc=gdb-patches@sources.redhat.com \
    --cc=jimb@red-bean.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