[MPlayer-cvslog] CVS: main configure, 1.1044, 1.1045 Makefile, 1.329, 1.330

[Diego Biurrun]

On Sat, Aug 20, 2005 at 09:30:04PM -0400, Rich Felker wrote:
> On Sat, Aug 20, 2005 at 06:46:36PM +0200, Diego Biurrun wrote:
> > 
> > BTW, can we all try to keep cvs log messages a bit more descriptive?
> > Things like 10l may be fun and quick to type and the problem may even be
> > obvious to those in the know, but don't forget that many people are
> > around to learn and explanations help them.
> 
> Are you talking about my v4l2 commit? IMO the error is obvious from
> the (one-line) patch. Dividing int/int to get a float does not work.
> Normally I say a lot more than just "10l" (or whatever amount) except
> when the error is obvious to anyone who knows C, or a typo, or a wrong
> numbers, etc.

Let's say that it was inspired by that commit, but I'm talking in
general, there are many more examples.  Yes, it will be obvious to those
in the know, but if you look at the output of 'cvs log' then 10l is not
descriptive unless you see the diff as well, while something like
"int/int does not produce a float value" is...

Diego