From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 23075 invoked by alias); 30 Oct 2004 03:28:34 -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 23068 invoked from network); 30 Oct 2004 03:28:33 -0000 Received: from unknown (HELO capitol.mail.pas.earthlink.net) (207.217.120.180) by sourceware.org with SMTP; 30 Oct 2004 03:28:33 -0000 Received: from ip216-26-76-19.dsl.du.teleport.com ([216.26.76.19] helo=stray.canids) by capitol.mail.pas.earthlink.net with esmtp (Exim 3.33 #1) id 1CNjup-0004wS-00; Fri, 29 Oct 2004 20:28:31 -0700 Received: from stray.canids (localhost.localdomain [127.0.0.1]) by stray.canids (Postfix) with ESMTP id DB2D2501085; Fri, 29 Oct 2004 20:28:28 -0700 (PDT) From: Felix Lee To: Andrew Cagney Cc: gdb-patches@sources.redhat.com Subject: Re: backtrace changes current source location References: <20041026075115.4A2C354AAB5@stray.canids> <20041026132924.GA26886@nevyn.them.org> <20041026150127.6ED3E54AAB5@stray.canids> <417FDC11.7060700@gnu.org> <20041028005157.259D34E8F0A@stray.canids> <41825FD5.1030607@gnu.org> <20041029220723.CCE45501085@stray.canids> <4182D9AD.5010003@gnu.org> In-Reply-To: <4182D9AD.5010003@gnu.org> on Fri, 29 Oct 2004 20:00:45 EDT from Andrew Cagney Date: Sat, 30 Oct 2004 03:28:00 -0000 Message-Id: <20041030032828.DB2D2501085@stray.canids> X-SW-Source: 2004-10/txt/msg00489.txt.bz2 Andrew Cagney : > Should: > (gdb) info frame 5 > change things to frame 5's sal? it doesn't do that now. it's only when a line like #4 0x08048392 in main () at z1.c:8 gets printed, which happens at breakpoints, or when the user selects a frame with 'frame', and so forth. > MI and/or CLI should locally determine current-sal, and pass it down to > print_frame_info. ok, so an alternate implementation is to add a function set_current_sal_to_frame(), and most places that call print_frame_info() will also call set_current_sal_to_frame(). does that sound acceptable? that will be a slightly larger patch. --