Speex dev - Jul 2007 - Small bug fixed

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

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@xiph.org
> http://lists.xiph.org/mailman/listinfo/speex-dev
> 
>