[Mplayer-cvslog] CVS: main/DOCS/tech cvs-howto.txt,1.4,1.5

[Arpi]

Hi,

> > +   Note: if you had to put if(){ .. } over big (> 5 lines) code,
> > +   do NOT change the indent of the inner part (move it right) !
> 
> 	Did you notice this policy is in direct conflict with your
> "coding style for a particular modular shall be forever held
> sacrosanct" policy? And I'll continue to flout it. Have I ever told you
> that the single worst thing about your "coding style" is to declare
> arbitrary scopes in the middle of your code because you need a local
> variable there? And you don't even bother to indent the scopes. Sloppy...

I rarely use that and only with reason.
I rather insert { } when i need soem locals there instead of placing the
variables 20 pages above just to avoid { and }
anyway the old opendivx c postprocess code gots 50% faster just because i
moved soem local variables inside the loops and { } decreasing their scope
and increasing the possibility of placing them into registers by teh
compiler...


A'rpi / Astral & ESP-team

--
Developer of MPlayer, the Movie Player for Linux - http://www.MPlayerHQ.hu