[FFmpeg-devel] [PATCH] UltraSPARC VIS optimized yuv2rgb

[Måns Rullgård]

Diego Biurrun <diego at biurrun.de> writes:

> On Mon, Jul 16, 2007 at 10:18:35PM +0200, Balatoni Denes wrote:
>> 
>> Monday 16 July 2007 21:39-kor Diego Biurrun ezt ?rta:
>> > The rules say 4 spaces, please change your code to use 4 space
>> > indentation.
>> 
>> Okay, I have found the rule. This patch should be fine.
>> 
>> --- libswscale.old/yuv2rgb_vis.c	1970-01-01 01:00:00.000000000 +0100
>> +++ libswscale/yuv2rgb_vis.c	2007-07-16 22:16:16.000000000 +0200
>> @@ -0,0 +1,208 @@
>> +
>> +static short int  __attribute__((aligned(8))) sparc_coeffs[4*10]=
>> +{
>> +    596,  596,  596,  596, // 16*1.164*32
>> +    8266, 8266, 8266, 8266, //128*2.018*32
>> +    1602, 1602, 1602, 1602, //128*0.391*32
>> +    3330, 3330, 3330, 3330, //128*0.813*32
>> +    6537, 6537, 6537, 6537, //128*1.596*32
>> +    9535, 9535, 9535, 9535, //1.164*32*256
>> +    6660, 6660, 6660, 6660, //0.813*32*256
>> +    13074,13074,13074,13074, //1.596*32*256
>> +    16531,16531,16531,16531, //2.018*32*256
>> +    3203, 3203, 3203, 3203, //0.391*32*256
>> +};
>
> This could be vertically aligned.

Right-aligning the numbers would make sense here, even if that means
varying indentation.  I guess it could be thought of as indentation
followed by leading spaces... or something.

-- 
M?ns Rullg?rd
mans at mansr.com