[FFmpeg-cvslog] r17068 - in trunk/libavformat: audiointerleave.c audiointerleave.h

[Måns Rullgård]

Baptiste Coudurier <baptiste.coudurier at gmail.com> writes:

> Reimar D?ffinger wrote:
>> On Sun, Feb 08, 2009 at 03:09:34PM -0800, Baptiste Coudurier wrote:
>>> Besides, Im sorry but I still think the commit message cannot be
>>> separated from the diff, and I won't change my mind.
>> 
>> Well, it already is separated, svn log will only show the message and
>> the message is above and the patch below.
>> Also by your argument you could just always give an empty commit
>> message.
>
> The mail everybody get from -cvslog is not separated, everybody needing
> more information will search for the diff, and comment on -cvslog ml if
> wanted, therefore reading the whole mail.
> Repeating this information in the commit message only adds redundant
> information. That's what getting the diff in -cvslog ml is for, and is
> very useful.

I don't know about others, but I often search for things in the commit
history, and this is much easier when messages are clear.  Browsing
long history lists, looking for something vague (i.e. that cannot
easily be grepped for) is much easier if the first line of the commit
message gives enough information that I can skip most of them.  This
is not possible when say imprecise things like "make the function static".

-- 
M?ns Rullg?rd
mans at mansr.com