[FFmpeg-cvslog] r14684 - trunk/libavcodec/fraps.c

[Mike Melanson]

Michael Niedermayer wrote:
> On Sun, Aug 10, 2008 at 03:37:14PM -0700, Mike Melanson wrote:
>> Michael Niedermayer wrote:
>>> On Sun, Aug 10, 2008 at 08:15:38PM +0200, kostya wrote:
>>>> Author: kostya
>>>> Date: Sun Aug 10 20:15:38 2008
>>>> New Revision: 14684
>>>>
>>>> Log:
>>>> R and B components are stored as a differences to G component in Fraps v5.
>>>>
>>>> This fixes roundup issue 574.
>>> [...]
>>>> +                U = f->data[0][0 + i*3 + j*f->linesize[0]];
>>>> +                Y = f->data[0][1 + i*3 + j*f->linesize[0]];
>>>> +                V = f->data[0][2 + i*3 + j*f->linesize[0]];
>>>> +                R = Y + (int8_t)U;
>>>> +                G = Y;
>>>> +                B = Y + (int8_t)V;
>>>> +                f->data[0][0 + i*3 + j*f->linesize[0]] = R;
>>>> +                f->data[0][1 + i*3 + j*f->linesize[0]] = G;
>>>> +                f->data[0][2 + i*3 + j*f->linesize[0]] = B;
>>> f->data[0][0 + i*3 + j*f->linesize[0]] += f->data[0][1 + i*3 + j*f->linesize[0]];
>>> f->data[0][2 + i*3 + j*f->linesize[0]] += f->data[0][1 + i*3 + j*f->linesize[0]];
>> Would this affect endian-ness? 
> 
> i dont think so ...
> Its just a simplification ...

Actually, the question concerned the original change (14684); I was 
unclear. After Kostya's change, the decoder produces different output on 
little and big endian systems.

-- 
	-Mike Melanson