[FFmpeg-cvslog] r14068 - trunk/libavcodec/ra288.c

Benoit Fouet benoit.fouet
Tue Jul 8 11:46:41 CEST 2008


vitor wrote:
> Log:
> Write for loops in a more standard way. In my opinion, they are much more readable now.
>
> Modified: trunk/libavcodec/ra288.c
> ==============================================================================
> --- trunk/libavcodec/ra288.c	(original)
> +++ trunk/libavcodec/ra288.c	Fri Jul  4 21:55:58 2008
> @@ -40,22 +40,27 @@ typedef struct {
>  /* Decode and produce output */
>  static void decode(Real288_internal *glob, float gain, int cb_coef)
>  {
> -    unsigned int x, y;
> +    int x, y;
>      double sum, sumsum;
>      float buffer[5];
>  
> -    for (x=36; x--; glob->sb[x+5] = glob->sb[x]);
> +    for (x=35; x >= 0; x--)
> +        glob->sb[x+5] = glob->sb[x];
>  
> -    for (x=5; x--;) {
> +    for (x=4; x >= 0; x--) {
>          float *p1 = glob->sb+x;
>          float *p2 = glob->pr1;
>   

as x is used in the loops, I don't think the new loops are equivalent to
the old ones...

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




More information about the ffmpeg-cvslog mailing list