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

[Vitor Sessak]

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.

-Vitor


-------------- next part --------------
A non-text attachment was scrubbed...
Name: ra144_swap.diff
Type: text/x-patch
Size: 374 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20080513/1d9c6821/attachment.bin>