[FFmpeg-cvslog] r13123 - trunk/libavcodec/ra144.c

[Michael Niedermayer]

On Tue, May 13, 2008 at 11:05:37AM +0200, Vitor Sessak wrote:
> Benoit Fouet wrote:
>> Hi,
>> astrange wrote:
>>> Author: astrange
>>> Date: Sun May 11 20:43:11 2008
>>> New Revision: 13123
>>>
>>> Log:
>>> Replace some hardcoded swapping with FFSWAP.
>>> Patch by Vitor Sessak (vitor1001 gmail com)
>>>
>>>
>>> Modified:
>>>    trunk/libavcodec/ra144.c
>>>
>>> Modified: trunk/libavcodec/ra144.c
>>> ==============================================================================
>>> --- trunk/libavcodec/ra144.c	(original)
>>> +++ trunk/libavcodec/ra144.c	Sun May 11 20:43:11 2008
>>> @@ -359,7 +357,7 @@ static int eq(Real144_internal *glob, sh
>>>      unsigned int u;
>>>      short *sptr;
>>>      int *ptr1, *ptr2, *ptr3;
>>> -    int *bp1, *bp2, *temp;
>>> +    int *bp1, *bp2;
>>>       retval = 0;
>>>      bp1 = glob->buffer1;
>>> @@ -403,9 +401,7 @@ static int eq(Real144_internal *glob, sh
>>>          if ((u + 0x1000) > 0x1fff)
>>>              retval = 1;
>>>  -        temp = bp2;
>>> -        bp2 = bp1;
>>> -        bp1 = temp;
>>> +        FFSWAP(unsigned int *, bp1, bp2);
>>>      }
>>>   
>> am I missing something, or should that be int * instead of unsigned int * 
>> ?
>
> Indeed. Thanks for noticing. Patch attached.

ok

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No great genius has ever existed without some touch of madness. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20080513/d46b7dc0/attachment.pgp>