[FFmpeg-cvslog] r14148 - trunk/libavcodec/dsputil.h

[Diego Biurrun]

On Thu, Jul 10, 2008 at 09:23:04AM +0100, M?ns Rullg?rd wrote:
> Diego Biurrun <diego at biurrun.de> writes:
> 
> > On Thu, Jul 10, 2008 at 08:15:13AM +0200, kostya wrote:
> >> 
> >> Log:
> >> Those functions input length is a multiple of 16.
> >
> > Could you be more specific than "those" in your commit messages please?
> > If you would just say
> >
> >   The input length of the APE functions should be a multiple of 16.
> >
> > then your commit message makes sense without having a look at the diff.
> 
> This goes for a lot of commit messages, if not most of them.  Many
> projects have the convention of prefixing commit messages with the
> name of the subsystem/module.  In our case, that would be codec and
> format names for most commits.

Yes, it's a crying shame.

I must say that I fail to understand why some aspects of coding are
optimized beyond measure around here and the strictest discipline is
required from all committers for even minor details.  At the same time
the importance of commit messages is completely neglected.  But commit
messages are an invaluable aid for debugging code written by others or
written a long time ago...

Diego