Re: FLL change
R Parker <[email protected]>
| Newsgroups | gmane.comp.audio.jamin.devel |
|---|---|
| Message-ID | <[email protected]> |
Hi, I'm forwarding to JAMin so Steve catches it. I tested Sampo's change. It's hard to reach a strong position. I listened and nothing weird occured and am suggesting you commit because it fixes a problem that Sampo experiences when slamming the limiter. I've heard the problem but don't hit the limiter hard very often so it's never a big deal to me. Steve, I think you should commit the change. Once again, Sampo has another fix. Awesome! Ron --- Sampo Savolainen <[email protected]> wrote: > In fast_lookahead_limiter_1913.xml: > > 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); > > > Disclaimer: line numbers may not match. They are > from 0.4.14 originally, > but they might have changed when i did testing & > debugging. > > This may cause slight clipping due to very high and > fast peak transients > (fast an is only for a few samples), but this > clipping should be > virtually inaudible because the peaks which get past > the limiter are so > very tiny in length. > > -- > Sampo Savolainen <[email protected]> > > __________________________________________________ Do You Yahoo!? Tired of spam? Yahoo! Mail has the best spam protection around http://mail.yahoo.com ------------------------------------------------------------------------- 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