[MPlayer-dev-eng] simple Blu-ray playback [Patch]

Diego Biurrun diego at biurrun.de
Fri Jul 2 14:56:58 CEST 2010


On Fri, Jul 02, 2010 at 02:08:55PM +0200, Alexander Roalter wrote:
> On 02.07.2010 12:55, Diego Biurrun wrote:
> > On Fri, Jul 02, 2010 at 11:09:00AM +0200, Alexander Roalter wrote:
> >> +        mp_msg(MSGT_OPEN, MSGL_ERR, 
> >> +            "Cannot open file %s to get UK and DiscID\n", filename);
> > 
> > Indentation is messed up here, the function arguments should be
> > aligned.
> Aligned with first parameter. Should in this case all parameters be put
> in a separate line? à la
> mp_msg(a,
>        b,
>        c,
>        d)
> 
> or only do it as long as it fits within the first say 75 characters?

At your discretion; usually the latter looks better.

Diego



More information about the MPlayer-dev-eng mailing list