[MPlayer-dev-eng] [PATCH] ve_lavc cosmetics

Robert Swain robert.swain at gmail.com
Sun Feb 12 18:46:26 CET 2006


On Sun, 2006-02-12 at 18:35 +0100, Michael Niedermayer wrote:
> Hi

[...]

> iam against it
> 
> i agree with 
> * 4space indention
> * trailing whitespace removial
> * tab removial
> * i agree with moving "{" which are on their own line
> 
> i do not agree with
> * x=y / x = y changes
> * breaking lines > 80 columns
> * removing {} of if/for if theres just one statement inside
> * reindenting lavcopts_conf table, this serves no purpose
> 
> [...]

OK, I'll make another patch that does what you say. I would prefer
longer lines to breaking into < 80 columns personally but I thought it
was wanted. I'm glad it isn't.

Rob




More information about the MPlayer-dev-eng mailing list