This is the mail archive of the binutils@sources.redhat.com mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: [patch] MIPS32 is a real ISA, and MIPS ISA/CPU name/constant cleanup.


Doug Evans <dje@transmeta.com> writes:
> I have a style nit.
> The GNU coding standards say to not put a blank line
> between associated ChangeLog entries.
> 
>     Separate unrelated change log entries with blank lines.  When two
>     entries represent parts of the same change, so that they work together,
>     then don't put blank lines between them.  Then you can omit the file
>     name and the asterisk when successive entries are in the same file.

Oh, hmm.  I'd basically been sorting by file, on the theory that
obviously the whole thing was in some sense "related."  8-)  (I've
never really read the GNU coding standards, i've tried to pick up what
I can from the sources and what I see here.)


So, it should go something like:

	* file1 (fn1, fn2): Make it sing.
	(fn3): Supply two different song choices.
	* file2, file3: New files, containing audio samples

	* file1 (fn1, fn2): Make it dance.
	* file4 (fn3): Likewise.
	(fn4): Add dance steps.

?


chris

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]