From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 81946 invoked by alias); 18 Nov 2018 09:23:07 -0000 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 Received: (qmail 81936 invoked by uid 89); 18 Nov 2018 09:23:07 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-25.1 required=5.0 tests=AWL,BAYES_00,GIT_PATCH_0,GIT_PATCH_1,GIT_PATCH_2,GIT_PATCH_3,SPF_PASS autolearn=ham version=3.3.2 spammy=experienced, thirty, Hx-languages-length:1993, onerous X-HELO: jocasta.intra Received: from de.cellform.com (HELO jocasta.intra) (88.217.224.109) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Sun, 18 Nov 2018 09:23:05 +0000 Received: from jocasta.intra (localhost [127.0.0.1]) by jocasta.intra (8.15.2/8.15.2/Debian-8) with ESMTPS id wAI9Mwnq017956 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Sun, 18 Nov 2018 10:22:58 +0100 Received: (from john@localhost) by jocasta.intra (8.15.2/8.15.2/Submit) id wAI9Mwtu017955; Sun, 18 Nov 2018 10:22:58 +0100 Date: Sun, 18 Nov 2018 09:23:00 -0000 From: John Darrington To: Kevin Buettner Cc: gdb-patches@sourceware.org, John Darrington Subject: Re: [PATCH 1/2] GDB: S12Z: Add assertion Message-ID: <20181118092258.wmiltm7lfzj2cvnw@jocasta.intra> References: <20181112091721.28040-1-john@darrington.wattle.id.au> <20181117145928.726522e6@pinnacle.lan> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20181117145928.726522e6@pinnacle.lan> User-Agent: NeoMutt/20170113 (1.7.2) X-SW-Source: 2018-11/txt/msg00275.txt.bz2 On Sat, Nov 17, 2018 at 02:59:28PM -0700, Kevin Buettner wrote: > --- > gdb/s12z-tdep.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/gdb/s12z-tdep.c b/gdb/s12z-tdep.c > index 79f5772035..bd0bd7c001 100644 > --- a/gdb/s12z-tdep.c > +++ b/gdb/s12z-tdep.c > @@ -320,6 +320,7 @@ s12z_frame_cache (struct frame_info *this_frame, void **prologue_cache) > } > else > { > + gdb_assert (this_sp == this_sp_for_id); > /* The stack pointer of the prev frame is frame_size greater > than the stack pointer of this frame plus one address > size (caused by the JSR or BSR). */ > -- Could you please add a ChangeLog entry? Also, on the same topic, what happened to the ChangeLog entries for the initial commit / push of the s12z port? (I don't see them in the ChangeLog file.) Is it really necessary in 2018 to have ChangeLogs? After all, the same information is available by typing git log and/or git diff. If the answer is "yes", then how about generating it automatically using the git-log-to-changelog script which is available in gnulib ? Many other projects have gone over to this method. ChangeLogs are a dreaded nuisance. They are almost guaranteed to cause merge conflicts. The information they contain is redundant. Their formatting requirements are onerous. And in thirty years of software development, I don't think I've ever experienced a case where I've needed to refer to one. As for adding the assert, I suppose that you're just reminding the reader of this earlier assignment? ... this_sp_for_id = this_sp; Indeed. (I didn't see anything which might change either this_sp or this_sp_for_id along the code path leading to the assert.) I have some plans for future changes in this part of the code. J' -- Avoid eavesdropping. Send strong encrypted email. PGP Public key ID: 1024D/2DE827B3 fingerprint = 8797 A26D 0854 2EAB 0285 A290 8A67 719C 2DE8 27B3 See http://sks-keyservers.net or any PGP keyserver for public key.