[FFmpeg-devel] [PATCH] GSM 6.10 audio decoder

Diego Biurrun diego
Mon Jul 5 23:03:51 CEST 2010


On Mon, Jul 05, 2010 at 10:35:52PM +0200, Reimar D?ffinger wrote:
> On Mon, Jul 05, 2010 at 10:19:41PM +0200, Diego Biurrun wrote:
> >      for (i = 0; i < 8; i++)
> > -        rrp[i] = get_rrp((lar_prev[i] >> 2) + (lar     [i] >> 1) + (lar[i] >> 2));
> > +        rrp[i] = get_rrp((lar_prev[i] >> 2) + (lar[i] >> 1) + (lar[i] >> 2));
> 
> The 3 of those are supposed to be aligned.

I'm afraid you've missed the alignment :)

> --- libavcodec/gsmdec.c	(revision 0)
> +++ libavcodec/gsmdec.c	(revision 0)
> @@ -0,0 +1,316 @@
> +
> +    for (i = 0; i < 8; i++)
> +        rrp[i] = get_rrp((lar_prev[i] >> 2) + (lar_prev[i] >> 1) + (lar[i] >> 2));
> +
> +    for (i = 0; i < 8; i++)
> +        rrp[i] = get_rrp((lar_prev[i] >> 1) + (lar    [i] >> 1));
> +
> +    for (i = 0; i < 8; i++)
> +        rrp[i] = get_rrp((lar_prev[i] >> 2) + (lar     [i] >> 1) + (lar[i] >> 2));

Something is off here..

Diego



More information about the ffmpeg-devel mailing list