[FFmpeg-cvslog] r14267 - trunk/libavcodec/ra288.c

Diego Biurrun diego
Fri Jul 18 07:59:34 CEST 2008


On Fri, Jul 18, 2008 at 01:38:43AM +0200, Vitor Sessak wrote:
> Diego Biurrun wrote:
> > On Fri, Jul 18, 2008 at 12:42:23AM +0200, vitor wrote:
> >> Log:
> >> Simplify
> > 
> > Can we *please* have more descriptive commit messages?  How long can it
> > take you to explain *what* you simplified?
> 
> For such obvious cleanups I'm against spending more time thinking about 
> the commit message than doing the code changes (even more so as what is 
> "cleaner" is a matter of taste, so it is non trivial to explain why the 
> new code is better in a commit msg). But if you could suggest anything 
> better that I could copy-paste for those kind of clean-up commits, I'd 
> happily do so.

As a rule of thumb:

- Never ever use one-word commit messages,
- at least mention which function you simplified.

How hard can that be?  And how much extra time will that cost you?  10
seconds?  30 seconds?  You make it sound almost as if you would gladly
skip commenting your code if it would make you save time...

> -Vitor, who didn't want to engage in this flamewar...

Writing bad commit messages is not the way to avoid the flames,
obviously...

Diego




More information about the ffmpeg-cvslog mailing list