From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 21480 invoked by alias); 10 Apr 2002 13:27:20 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 21441 invoked from network); 10 Apr 2002 13:27:16 -0000 Received: from unknown (HELO cygnus.com) (205.180.230.5) by sources.redhat.com with SMTP; 10 Apr 2002 13:27:16 -0000 Received: from redhat.com (rtl.sfbay.redhat.com [205.180.230.21]) by runyon.cygnus.com (8.8.7-cygnus/8.8.7) with ESMTP id GAA20000; Wed, 10 Apr 2002 06:25:49 -0700 (PDT) Message-ID: <3CB43CB7.F7BBB223@redhat.com> Date: Wed, 10 Apr 2002 06:27:00 -0000 From: Fernando Nasser Organization: Red Hat Canada X-Accept-Language: en MIME-Version: 1.0 To: Keith Seitz CC: Jim Ingham , gdb-patches@sources.redhat.com Subject: Re: Nother little one, this time in varobj.c References: Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit X-SW-Source: 2002-04/txt/msg00397.txt.bz2 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