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

[Michael Niedermayer]

Hi

On Sun, Aug 21, 2005 at 11:30:26AM +0200, Torinthiel wrote:
> On Sun, Aug 21, 2005 at 10:26:08AM +0200, Michael Niedermayer wrote:
> > 
> > this is still not good, the commit log should at least contain
> > * high level description of what the change does (fixing fps calculation
> >   in setup_audio_buffer_sizes()) but actually thats still not good as it
> >   says nothing about the conequences of the wrong fps ...
> > * low level descrioption / diff summary like (int/int does not produce...)
> > * list of bug numbers / mailing list archive links which get fixed
> > * possible unwanted sideeffects of the change
> > * summary of any test/benchmarks done
> > * severity of the bug/change
> > * some list of tags from (cosmetic/crash-fix/rounding-fix/security-fix/
> >   spelling-fix/indention-fix/untested/new-feature/optimization)
> 
> Nice big list. Yes, it would be ideal to have this, but than it's a big
> PITA to do it. 

i know, it wasnt really meant to be serious, but some parts of it might be
interresting ...


[...]

> Usually it's just 'didn't work before, now works'. Maybe (I don't know
> if it's possible with CVS) there could be a form to fill, and then the

possible, see CVSROOT/rcsinfo
its also possible to check & reject messages, see CVSROOT/verifymsg
so we could detect and reject 10l commits if that is wanted

[...]
-- 
Michael