[MPlayer-dev-eng] [PATCH]VO_VDPAU, round 4

Diego Biurrun diego at biurrun.de
Thu Jan 22 09:30:11 CET 2009


On Thu, Jan 22, 2009 at 12:22:23AM +0000, Carl Eugen Hoyos wrote:
> Diego Biurrun <diego <at> biurrun.de> writes:
> 
> > > > inconsistent formatting,
> > > 
> > > Do you see one outside of uninit()?
> > 
> > Brace placement is often inconsistent (K&R style preferred),
> 
> This does only apply to function declarations, or am I wrong (I could not find a
> function that did not have its "{" on a new line)?

K&R style is more than just function declarations of course.

Specifically, it places a space after flow control keywords (if/while/..)
and the braces on the same line, i.e.

    if (condition) {
        statement1;
        statement2;
    }

Just try running your file through indent with the proper options to
see the difference or look at the first hit Google produces for K&R
indent style.

Diego



More information about the MPlayer-dev-eng mailing list