[FFmpeg-cvslog] r24066 - trunk/libavfilter/vf_pad.c

[Måns Rullgård]

Stefano Sabatini <stefano.sabatini-lala at poste.it> writes:

> On date Tuesday 2010-07-06 11:26:16 +0100, M?ns Rullg?rd wrote:
>> stefano <subversion at mplayerhq.hu> writes:
>> 
>> > Author: stefano
>> > Date: Tue Jul  6 00:33:02 2010
>> > New Revision: 24066
>> >
>> > Log:
>> > Factorize out some code and implement the fill_line_with_color()
>> > function. Allow to share code with the pending color source.
>> >
>> > +        color[RED  ] = RGB_TO_Y(rgba_color[0], rgba_color[1], rgba_color[2]);
>> > +        color[GREEN] = RGB_TO_U(rgba_color[0], rgba_color[1], rgba_color[2], 0);
>> > +        color[BLUE ] = RGB_TO_V(rgba_color[0], rgba_color[1], rgba_color[2], 0);
>> > [...]
>> > -        pad->color[0] = RGB_TO_Y_CCIR(rgba_color[0], rgba_color[1], rgba_color[2]);
>> > -        pad->color[1] = RGB_TO_U_CCIR(rgba_color[0], rgba_color[1], rgba_color[2], 0);
>> > -        pad->color[2] = RGB_TO_V_CCIR(rgba_color[0], rgba_color[1], rgba_color[2], 0);
>> 
>> Was dropping the _CCIR intentional?  It's breaking the tests.
>
> WTF thanks for having a look at it, now that was completely
> unintentional I suppose I messed it up... feel free to fix it, or I'll
> do it this night.

Fixed.

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