This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: Nother little one, this time in varobj.c
- From: Fernando Nasser <fnasser at redhat dot com>
- To: Keith Seitz <keiths at redhat dot com>
- Cc: Jim Ingham <jingham at apple dot com>, gdb-patches at sources dot redhat dot com
- Date: Wed, 10 Apr 2002 09:23:03 -0400
- Subject: Re: Nother little one, this time in varobj.c
- Organization: Red Hat Canada
- References: <Pine.GSO.4.33.0204092335270.2411-100000@cse.cygnus.com>
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...)
>
The frames code was already there when I converted it to gdbtk-varobj.c,
so it was added in between. It seemed to work though.
--
Fernando Nasser
Red Hat Canada Ltd. E-Mail: fnasser@redhat.com
2323 Yonge Street, Suite #300
Toronto, Ontario M4P 2C9