[MPlayer-cvslog] CVS: main/libmpcodecs vf_scale.c,1.57,1.58

Nico Sabbi nsabbi at tiscali.it
Sun Mar 6 22:44:00 CET 2005


Oded Shimon wrote:

>On Sunday 06 March 2005 23:15, Nico Sabbi CVS wrote:
>  
>
>>+      vf->priv->w = -(-vf->priv->w & ~8);
>>    
>>
>
>Why didn't you change this to
>vf->priv->w += 8;
>after all?
>As far as I can tell, it's completely identical, and much more readable...
>Also personally in the 'if', I would've done '<= -8' instead of the mess that 
>is over there.
>
>Should I submit a cosmetics patch? :)
>
>  
>

no, I'll let Reimar decide and fix it if he wants, I'm sure he will, 
won't you Reimar?  :)




More information about the MPlayer-cvslog mailing list