From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 24369 invoked by alias); 7 Oct 2008 03:49:19 -0000 Received: (qmail 24360 invoked by uid 22791); 7 Oct 2008 03:49:18 -0000 X-Spam-Check-By: sourceware.org Received: from rock.gnat.com (HELO rock.gnat.com) (205.232.38.15) by sourceware.org (qpsmtpd/0.31) with ESMTP; Tue, 07 Oct 2008 03:48:43 +0000 Received: from localhost (localhost.localdomain [127.0.0.1]) by filtered-rock.gnat.com (Postfix) with ESMTP id A50402A96B3; Mon, 6 Oct 2008 23:48:41 -0400 (EDT) Received: from rock.gnat.com ([127.0.0.1]) by localhost (rock.gnat.com [127.0.0.1]) (amavisd-new, port 10024) with LMTP id iGRwYxXl+Drq; Mon, 6 Oct 2008 23:48:41 -0400 (EDT) Received: from joel.gnat.com (localhost.localdomain [127.0.0.1]) by rock.gnat.com (Postfix) with ESMTP id 69C622A96AB; Mon, 6 Oct 2008 23:48:41 -0400 (EDT) Received: by joel.gnat.com (Postfix, from userid 1000) id BE16BE7ACD; Mon, 6 Oct 2008 20:48:40 -0700 (PDT) Date: Tue, 07 Oct 2008 03:49:00 -0000 From: Joel Brobecker To: Michael Snyder Cc: "gdb-patches@sourceware.org" , Pedro Alves , teawater Subject: Re: [RFA] Reverse Debugging, 1/5 Message-ID: <20081007034840.GE28138@adacore.com> References: <48E3CCB6.4060501@vmware.com> <20081006203021.GA21853@adacore.com> <48EA7C75.7070703@vmware.com> <20081006211131.GA26663@caradoc.them.org> <48EA8065.9070001@vmware.com> <20081006212504.GB31085@caradoc.them.org> <48EA868E.3070404@vmware.com> <20081006222258.GF21853@adacore.com> <48EAB0AD.30906@vmware.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <48EAB0AD.30906@vmware.com> User-Agent: Mutt/1.4.2.2i Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2008-10/txt/msg00199.txt.bz2 > Before I actually implement this, let me see if we're all > on the same page (Daniel, Joel, Pedro...) My own view was a little simpler: Delete the target_set_execdir method, and replace all the calls with a reference to the infrun global. I would keep the target_set_execdir more or less as is; otherwise, you'll need some kind of observer to notice when the execdir changes. The "to_can_go_backwards" is an interesting idea, but in my opinion only makes sense if the target_set_execdir method is removed. Otherwise, we can treat target_set_execdir == NULL as cannot-go-backwards. These are just my opionions, I don't feel strongly about them. -- Joel