[MPlayer-cvslog] r25100 - trunk/libaf/af_scaletempo.c

[Reimar Döffinger]

Hello,
On Sun, Nov 18, 2007 at 07:49:24PM +0200, Uoti Urpala wrote:
> On Sun, 2007-11-18 at 18:28 +0100, reimar wrote:
> >      ppc = s->buf_pre_corr;
> > -    for (i=s->num_channels; i<s->samples_overlap; i++) {
> > +    ppc += s->samples_overlap - s->num_channels;
> > +    ps  += s->samples_overlap - s->num_channels;
> > +    i  = -(s->samples_overlap - s->num_channels);
> > +    do {
> >        corr += ( *ppc++ * *ps++ ) >> s->shift_corr;
> > -    }
> > +    } while (++i < 0);
> 
> The added += lines for ppc and ps look wrong.

Arg. Crap. Copy&Paste bug.
No, it is supposed to be ppc[i] and ps[i].
I just hope this happened after benchmarking...

Greetings,
Reimar Döffinger