[FFmpeg-devel] [PATCH] [3/3] SIPR 16k decoder

Ronald S. Bultje rsbultje
Fri Jan 15 01:30:16 CET 2010


Hi,

On Thu, Jan 14, 2010 at 7:25 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> On Thu, Jan 14, 2010 at 07:19:47PM -0500, Ronald S. Bultje wrote:
>> On Thu, Jan 14, 2010 at 7:05 PM, Vitor Sessak <vitor1001 at gmail.com> wrote:
>> > The rule I use is: always use floats, unless when you need to call
>> > ff_acelp_lspd2lpc(), which takes doubles as arguments (and also to
>> > avoiding converting buffers to double or back).
>>
>> So let's change ff_acelp_lspd2lpc()? Why does it use doubles?
>
> dont forget testing PSNR & with yor ears if PSNR differs
>
> i remember this double vs float discussion and the last time the awnser IIRC
> was that floats are not accurate enough. I dont want to see the floats are
> faster on my system so change it discussion repeat until testing is forgotten
> and we put a possibly hard to debug bug in

My point is more or less that *ALL* voice decoders should use doubles
everywhere. The 16k code uses floats, takes a cos(f), casts it to a
double and suddenly we have a double. That's not right. It should've
used doubles from the beginning if the goal was to do a double.

And why on earth are LPCs in float if LSPs are in double?

Ronald



More information about the ffmpeg-devel mailing list