[MPlayer-cvslog] r25779 - in trunk/libswscale: rgb2rgb_template.c swscale-example.c swscale_altivec_template.c

Benoit Fouet benoit.fouet at purplelabs.com
Mon Jan 21 09:48:02 CET 2008


Diego Biurrun wrote:
> On Thu, Jan 17, 2008 at 12:07:27PM +0100, benoit wrote:
>   
>> Log:
>> Remove some useless parentheses.
>>
>> --- trunk/libswscale/rgb2rgb_template.c	(original)
>> +++ trunk/libswscale/rgb2rgb_template.c	Thu Jan 17 12:07:27 2008
>> @@ -1621,7 +1621,7 @@ static inline void RENAME(yuvPlanartoyuy
>>  #endif
>> -        if ((y&(vertLumPerChroma-1))==(vertLumPerChroma-1) )
>> +        if ((y&(vertLumPerChroma-1)) == vertLumPerChroma-1)
>>     
>
> While you're at it: I think spaces around operators make things
> considerably more readable, i.e.
>
>   if ((y & (vertLumPerChroma - 1)) == vertLumPerChroma - 1)
>
>   

I added the spaces around the '==' operator, which seemed enough to me...
now, if you want to add extra spaces, please do :)

-- 
Ben
Purple Labs S.A.
www.purplelabs.com



More information about the MPlayer-cvslog mailing list