From: Jim Ingham <jingham@apple.com>
To: Daniel Jacobowitz <drow@false.org>
Cc: Vladimir Prus <ghost@cs.msu.su>, gdb@sources.redhat.com
Subject: Re: -var-update and address changes
Date: Fri, 14 Apr 2006 20:03:00 -0000 [thread overview]
Message-ID: <DB0425AC-0CCB-448C-BC15-7E6644C6CB0B@apple.com> (raw)
In-Reply-To: <20060414131549.GC12955@nevyn.them.org>
On Apr 14, 2006, at 6:15 AM, Daniel Jacobowitz wrote:
> On Wed, Apr 12, 2006 at 04:04:36PM +0400, Vladimir Prus wrote:
>> I'm running into what looks like a bug in -var-update. Basically,
>> I create a
>> varobj with "&variable", then I enter a function that has a
>> variable with
>> the same name, and -var-update does not report anything.
>
> I think we've concluded that this is a feature, not a bug...
>
>> The value of "&i" changes, but -var-update does not report this.
>> This can be
>> explained by the fact that varobjs are "bound" to the stack frame
>> where
>> they were created, but that "binding" is not mentioned in
>> documentation.
>
> ... and adding this to the manual would be welcome.
The docs are there, just not altogether clear. They say:
The frame under which the expression should be evaluated can be
specified by @var{frame-addr}. A @samp{*} indicates that the current
frame should be used.
The last sentence means that the varobj will be bound to whatever the
current frame is. It can be read to mean "it's re-evaluated in
whatever the current frame happens to be when you update it" - maybe
that's how Vlad understood it. So you just need to make this a
little clearer.
Note as an aside, that we had to add another varobj type which is
evaluated in the selected frame, whatever that happens to be. That
was useful for a general "variable inspector" window. People wanted
to put some expression there, and have it re-evaluated in the topmost
frame whenever they stopped. So we added that functionality. But
that is clearly distinct from what the "*" varobj type is supposed to
mean.
>
>> The problem I'm trying to solve is this:
>> 1. In some frame, I create varobj for 'i'.
>> 2. After continue, I see that there's local variable 'i', and I
>> want to find
>> out if previously-created varobj can be used to show this local
>> 'i', or if
>> I should create new 'i'.
>
> OK, so, you need to map this to a frame. Apple outputs the frame
> address. I think this is pretty nasty; frame IDs ought to be
> opaque in
> the interface (e.g. in case we change their contents again)... Jim,
> what does your front end do with the frame IDs? Is there anything
> besides testing for equality? If there is, we can use unique opaque
> identifiers instead; personally I'm much happier with that. The
> best unique identifier might in fact be the contents of the frame
> ID; I'm just trying to define how clients are allowed to interpret
> it, hopefully not at all.
The fp part is just used as part of the frame fingerprint. The pc is
used for other purposes of course (for instance if the disassembly
window is open), but the fp part is just used to check whether the
frame needs to be refreshed. I have no problem with it being opaque.
>
>> Can somebody suggest the right fix? So far, I think that the simplest
>> approach is to make gdb print stack address of current frame, like
>> is done
>> on the Apple branch:
>>
>> 553^done,stack=[frame=
>> {level="0",addr="0x00003db0",fp="0xbffff2c0",......
>>
>> That way, frontend can deal with the issue of frame stacks
>> themself, and
>> -var-update will be only used when single-stepping inside a given
>> frame.
>> Will patches to implement this be welcome?
>
> I guess. It seems reasonable.
>
> The MI working group list is now open; should we move this sort of
> conversation there?
That seems appropriate.
Jim
next prev parent reply other threads:[~2006-04-14 19:48 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-04-12 16:02 Vladimir Prus
2006-04-12 18:25 ` Jim Ingham
2006-04-13 9:25 ` Vladimir Prus
2006-04-13 17:31 ` Jim Ingham
2006-04-14 13:25 ` Daniel Jacobowitz
2006-04-14 20:03 ` Jim Ingham [this message]
2006-04-14 20:09 ` Daniel Jacobowitz
2006-04-14 20:27 ` Jim Ingham
2006-04-14 21:37 ` Daniel Jacobowitz
2006-04-17 6:18 ` Vladimir Prus
2006-04-17 9:02 ` Mark Kettenis
2006-04-17 10:54 ` Vladimir Prus
2006-05-02 12:50 ` Vladimir Prus
2006-05-02 13:14 ` Nick Roberts
2006-05-02 13:41 ` Vladimir Prus
2006-05-02 17:23 ` Jim Ingham
2006-05-03 6:03 ` Vladimir Prus
[not found] ` <20060504145046.GA32605@nevyn.them.org>
2006-05-04 15:12 ` Vladimir Prus
2006-05-04 15:13 ` Daniel Jacobowitz
2006-05-05 6:25 ` Vladimir Prus
2006-05-05 15:02 ` Daniel Jacobowitz
2006-04-16 15:52 ` 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=DB0425AC-0CCB-448C-BC15-7E6644C6CB0B@apple.com \
--to=jingham@apple.com \
--cc=drow@false.org \
--cc=gdb@sources.redhat.com \
--cc=ghost@cs.msu.su \
/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