[patch, fortran] Merge whitespace fixes from fortran-experiments branch

Brooks Moses brooks.moses@codesourcery.com
Thu Jun 7 09:03:00 GMT 2007


Steve Kargl spent quite a lot of time fixing whitespace problems with 
the early versions of the ISO C binding implementation on the 
fortran-experiments branch.  When I was reviewing the recent branch diff 
file, I noticed that he did a sufficiently good job of this that he also 
fixed quite a lot of problems that predated the ISO C binding work, and 
thus also exist on the trunk.

By "quite a lot", I mean 2500 lines of patch file. (!)

In my opinion, it would be useful to go ahead and merge those whitespace 
fixes to the trunk now, independently of the rest of the branch merging 
process.  That should make it easier to review the substantive changes 
on the branch when we get to that point.

Attached is a patch for trunk containing these whitespace fixes -- and, 
in particular, only the whitespace fixes that are sufficiently far away 
from the ISO C binding changes to avoid producing merge conflicts later. 
  I've build-tested it on powerpc-apple-darwin8.9.0, and am currently 
regression-testing it.  Ok to commit (assuming it passes reg-testing)?

Steve, what's your preference for the name on the changelog -- am I 
correct in putting your name on it?  (Or would you prefer that I take 
the blame in case something goes wrong? :) )

- Brooks

---------------------------------------------------------------
2007-06-06  Steven G. Kargl  <kargl@gcc.gnu.org>

	decl.c: Miscellaneous whitespace fixes.
	expr.c: Likewise.
	gfortran.h: Likewise.
	interface.c : Likewise.
	io.c: Likewise.
	match.c: Likewise.
	match.h: Likewise.
	module.c: Likewise.
	parse.c: Likewise.
	resolve.c: Likewise.
	symbol.c: Likewise.
	trans-array.c: Likewise.
	trans-common.c: Likewise.
	trans-decl.c: Likewise.
	trans-intrinsic.c: Likewise.
	trans-io.c: Likewise.
	trans-stmt.c: Likewise.
	trans-types.c: Likewise.

---------------------------------------------------------------

-------------- next part --------------
A non-text attachment was scrubbed...
Name: icb_fortran_formatonly.diff
Type: text/x-patch
Size: 77705 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/fortran/attachments/20070607/13f2b568/attachment.bin>


More information about the Fortran mailing list