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

Diego Biurrun diego at biurrun.de
Tue Feb 14 01:03:43 CET 2006


On Mon, Feb 13, 2006 at 12:39:21AM +0100, Michael Niedermayer wrote:
> 
> On Sun, Feb 12, 2006 at 03:51:51PM -0500, Rich Felker wrote:
> [...]
> 
> > The idea that whitespace should conform to a single 'coding style'
> > rule is IMO very misguided. The purpose of whitespace is to break the
> > logical units (logical in terms of how humans think about it, not how
> > the machine things) up in a way that makes them easier to read. This
> > depends on the semantics of the code, not the syntax.
> 
> yes, i fully agree
> here are a few more examples which would be literally obfuscated if
> they where are automatically reformated

I don't think I was expressing myself very clearly..

I was not proposing to add spaces regardless of context, just to be
consistent when adding spaces around operators like

var1=value;
var2 = value;

and of course prettyprinted columns should remain..

Diego




More information about the MPlayer-dev-eng mailing list