On Mon, Apr 18, 2011 at 18:08, Jim Meyering <j...@meyering.net> wrote: > I've rebased and divided/reordered these changes as you suggested. > Here are the fortran parts. I'll post the other parts separately. > Parts 1 and 3 are manual. Part 2 is the big mechanical change with > two manual adjustments: > > [PATCH 1/3] gfortran: remove cpp definition of free, ... > [PATCH 2/3] convert each use of gfc_free (p) to free (p) > [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. 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. (Why we, in the age of non-sucky version control, persist in keeping manual changelog files is beyond me..) With these changes, Ok for trunk. Thanks a lot for the contribution! -- Janne Blomqvist