[MPlayer-dev-eng] Smallest possible patch, or reindent if logic flow is changed?

Felix Buenemann atmosfear at users.sourceforge.net
Mon Nov 26 15:28:38 CET 2001


On Monday, 26. November 2001 10:36, you wrote:
> Hi,
>
> My patch adds an "else { }" around a biggish chunk of code.
> Do you want that code indented to reflect the logic flow change - which
> leads to a biggish patch - or left as it is to make a more readable patch
> (I suppose the indentation can be fixed later).
>
if you have large chunks eave the indenting and so something like that:
else { /* whatever_this_does */
[code]
} /* !whatever_this_does */

it keeps stuff readable.
> Thanks,
> Steve
>

-- 
Best Regards,
	Atmos
____________________________________________
- MPlayer Developer - http://mplayerhq.hu/ -
____________________________________________



More information about the MPlayer-dev-eng mailing list