[MPlayer-dev-eng] [PATCH] libbs2b: new api
Diego Biurrun
diego at biurrun.de
Tue Apr 21 10:53:21 CEST 2009
On Tue, Apr 21, 2009 at 12:33:47AM +0400, Andrew Savchenko wrote:
>
> On Monday 20 April 2009, Uoti Urpala wrote:
> > On Mon, 2009-04-20 at 22:45 +0400, Andrew Savchenko wrote:
>
> > + else
> > + if (!strcmp(s->profile,"cmoy"))
> >
> > "else if" should be on one line.
>
> No, no, no. The idea is to keep all if statements at the same
> alignment level, otherwise it will be mess to read. Currently they
> have nice and easy-to-understand-the-code indentation. Statements
> are logically equal and there is no reason to accent one of them.
> You may consider this as swith statement for strings.
>
> Yes, I know this is not strict K&R indentation but sometimes logics
> should override common rules.
I'm with Uoti on this one, just use standard K&R style.
> --- DOCS/man/en/mplayer.1 (revision 29211)
> +++ DOCS/man/en/mplayer.1 (working copy)
> @@ -5067,11 +5067,27 @@
> +.RS
> +If fcut or feed options are specified together with profile, they
a profile
Diego
More information about the MPlayer-dev-eng
mailing list