[Speex-dev] Small bug fixed

Jean-Marc Valin jean-marc.valin at usherbrooke.ca
Thu Jul 5 07:44:48 PDT 2007


Thanks, I'll apply that (poke me again if I forget). I tend to never
divide by a constant because it's slow, but here I forgot about the
overflow. As for the 44.1 kHz rate, it's perfectly fine with the
preprocessor. It's the codec that needs 8/16/32 kHz to work well.

	Jean-Marc

Dmitry Yakimov a écrit :
> Hi,
> 
> It is better to replace this line in function filterbank_new:
> max_mel = toBARK(EXTRACT16(MULT16_16_Q15(QCONST16(.5f,15),sampling)));
> 
> to
> 
> max_mel = toBARK(EXTRACT16(sampling/2));
> 
> It gives the same but it seems to be faster and avoids overflow on 44100
> kHz that prevents denoiser to process 44100 streams. (Yes I know that
> Speex should not pack 44100 streams but it does now and I use it).
> 
> Best Regards,
> Dmitry Yakimov
> 
> 
> _______________________________________________
> Speex-dev mailing list
> Speex-dev at xiph.org
> http://lists.xiph.org/mailman/listinfo/speex-dev
> 
> 


More information about the Speex-dev mailing list