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

[Benoit Fouet]

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 * ?

-- 
Benoit Fouet
Purple Labs S.A.
www.purplelabs.com