From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 8118 invoked by alias); 18 Apr 2011 16:32:14 -0000 Received: (qmail 8017 invoked by uid 22791); 18 Apr 2011 16:32:13 -0000 X-SWARE-Spam-Status: No, hits=-1.7 required=5.0 tests=AWL,BAYES_00,T_RP_MATCHES_RCVD X-Spam-Check-By: sourceware.org Received: from mx.meyering.net (HELO mx.meyering.net) (82.230.74.64) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Mon, 18 Apr 2011 16:31:57 +0000 Received: by rho.meyering.net (Acme Bit-Twister, from userid 1000) id 953796021D; Mon, 18 Apr 2011 18:31:55 +0200 (CEST) From: Jim Meyering To: Janne Blomqvist Cc: "Joseph S. Myers" , gcc-patches@gcc.gnu.org, fortran@gcc.gnu.org Subject: Re: [PATCH v3] Re: avoid useless if-before-free tests In-Reply-To: (Janne Blomqvist's message of "Mon, 18 Apr 2011 19:06:57 +0300") References: <87zkp9zmq0.fsf@rho.meyering.net> <877hc9r8w6.fsf_-_@rho.meyering.net> <877hc9pkhp.fsf_-_@rho.meyering.net> <87ei5w8nt7.fsf@rho.meyering.net> <87ei4z7g2i.fsf@rho.meyering.net> Date: Mon, 18 Apr 2011 16:39:00 -0000 Message-ID: <87wrir5xms.fsf@rho.meyering.net> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: quoted-printable Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org X-SW-Source: 2011-04/txt/msg01414.txt.bz2 Janne Blomqvist wrote: > On Mon, Apr 18, 2011 at 18:08, Jim Meyering wrote: >> I've rebased and divided/reordered these changes as you suggested. >> Here are the fortran parts. =A0I'll post the other parts separately. >> Parts 1 and 3 are manual. =A0Part 2 is the big mechanical change with >> two manual adjustments: >> >> =A0[PATCH 1/3] gfortran: remove cpp definition of free, ... >> =A0[PATCH 2/3] convert each use of gfc_free (p) to free (p) >> =A0[PATCH 3/3] gfortran: remove now-unused definition of gfc_free > > Thanks. The patches themselves are ok, however the changelog parts > should conform to the layout used otherwise for changelog entries (GNU > ChangeLog format, IIRC). The commit logs themselves that you have now > used are, IMHO, more informative and contain less clutter, but alas.. > > Note that it's recommended and usually easiest to provide changelog > entries separately, not as diffs from the ChangeLog file. Ok. I interpret that as meaning I should add a line for each file/function affected by the large mechanical change in part 2. I will append that to the existing ChangeLog entry. Thanks to Ralf's vc-chlog, that will be automatic: http://git.sv.gnu.org/cgit/vc-dwim.git/tree/README > Then when > you commit you insert the entry. This avoids the problem of frequent > conflicts as the top of the changelog file changes for every commit. If you work with the gcc repository using git, it's easy to rebase even pesky ChangeLog diffs once you hook up Bruno Haible's git-merge-changelog: http://git.sv.gnu.org/cgit/gnulib.git/tree/lib/git-merge-changelog.c If you'd like me to change the other two ChangeLog entries or any commit log message, please let me know. > (Why we, in the age of non-sucky version control, persist in keeping > manual changelog files is beyond me..) I couldn't agree more ;-) Even in projects without a VC'd ChangeLog file, I still keep one (and build up each ChangeLog entry as usual) and use vc-dwim to automatically cross-check any change I make. It regularly warns me about a changed file that's listed in ChangeLog but for which my editor has unsaved changes. > With these changes, Ok for trunk. Thanks a lot for the contribution! Thanks for the review.