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

Michael Niedermayer michaelni
Fri Jan 15 01:25:51 CET 2010


On Thu, Jan 14, 2010 at 07:19:47PM -0500, Ronald S. Bultje wrote:
> Hi,
> 
> 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 

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have never wished to cater to the crowd; for what I know they do not
approve, and what they approve I do not know. -- Epicurus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100115/26abb70f/attachment.pgp>



More information about the ffmpeg-devel mailing list