[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