Re: FLL change

On Mon, Jul 31, 2006 at 07:29:33 +0200, Robert Jonsson wrote:
> On Saturday 29 July 2006 21:27, Sampo Savolainen wrote:
> > The patch is really quite simple. This is done to
> > fast_lookahead_limiter_1913.xml in swh-plugins (0.4.14)
> >
> > line 89-90 (old code):
> >     const float this_delta = (max / chunks[p] - atten) /
> >                              ((float)i * fs * 0.0005f + 1.0f);
> >
> >
> > line 89-90 (new code):
> >     const float this_delta = (max / chunks[p] - atten) /
> >                              ((float)(i+1) * fs * 0.0005f + 1.0f);
> >                                      ^^^^^
> 
> Thanks a bunch for this fix, as far as I can tell the dreadful pumping is 
> gone :-)
> 
> Steve, I hope you can verify that this is correct and apply it to the sources.

Yes, it's correct, Sampo and I worked through it some time ago, and it
looked like the original (float)i was a mistake, but it's good to have
that confirmed by listening tests.

- Steve

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV