[MPlayer-cvslog] CVS: main mencoder.c,1.278,1.279

[Diego Biurrun]

On Wed, Apr 06, 2005 at 06:46:03PM +0200, Dominik 'Rathann' Mierzejewski wrote:
> On Wednesday, 06 April 2005 at 15:19, Diego Biurrun wrote:
> 
> > Granted, the CVS policy is ambiguous about this, but I never interpreted it
> > to forbid this kind of changes, have you?  Why do you oppose this?
> 
> ...the CVS policy is "do not mix cosmetic changes with functional changes".
> So, to follow it to the letter, you'd make two commits instead of one.

My interpretation of the above sentence is: Don't commit something where
one hunk is just cosmetic indentation changes and another is functional
changes.  I never interpreted it to forbid reindenting a line that is
being changed anyway.

Diego