This is the mail archive of the gdb-patches@sources.redhat.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Nother little one, this time in varobj.c


Keith,

Yeah, I added the ability to make the frames change (Syd thought that 
variables in the watch window should re-evaluate themselves every time 
that you changed the stack frame in the debugger - apparently that's 
what VC++ does).  But I added all this on 03-2000, and the 
reinit_frame_cache is already there in the initial import into 
sourceware.  Unfortunately, the sourceware repository history only goes 
back to Feb. 2000, and this was all checked in before that.  If you can 
look in the cygnus repository, you might at least be able to figure out 
who should remember why this is there - not that this actually helps 
that much...

Jim

On Tuesday, April 9, 2002, at 11:37  PM, Keith Seitz wrote:

> On Tue, 9 Apr 2002, Jim Ingham wrote:
>
>> BTW, I am not sure why it is necessary to call reinit_frame_cache here.
>> Keith, do you remember why this was necessary?  It is inefficient,
>> especially if you are evaluating a bunch of variables that are fairly
>> high up on the stack.  But since I don't remember why this was done, I
>> am reluctant to just change it outright...
>
> I haven't a clue, actually. When I originally wrote varobj for Insight, 
> it
> did not allow varobjs to change frames dynamically, as it does now.
> Presumably, this was all added at the same time. (Or I've forgotten all
> about it...)
>
> Keith
>
>
>
--
Jim Ingham                                   jingham@apple.com
Developer Tools - gdb
Apple Computer


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]