[FFmpeg-devel] [PATCH 4/4] mpegaudio_tablegen: Don't use llrint

Michael Niedermayer michaelni at gmx.at
Wed Jan 16 02:39:17 CET 2013


On Tue, Jan 15, 2013 at 08:36:05PM -0500, Derek Buitenhuis wrote:
> On 15/01/2013 8:16 PM, Michael Niedermayer wrote:
> > do these occur with pow() too or just powf() ?
> > in powf the 1.0/3  is at a much lower precission compared to a ideal
> > cbrtf()
> 
> $ diff -ur ./pre-patch/libavcodec/mpegaudio_tables.h ./powf-patch/libavcodec/mpegaudio_tables.h | diffstat
>  mpegaudio_tables.h |10042 ++++++++++++++++++++++++++---------------------------
>  1 file changed, 5021 insertions(+), 5021 deletions(-)
> 
> $ diff -ur ./pre-patch/libavcodec/mpegaudio_tables.h ./pow-patch/libavcodec/mpegaudio_tables.h | diffstat
>  mpegaudio_tables.h |10348 ++++++++++++++++++++++++++---------------------------
>  1 file changed, 5174 insertions(+), 5174 deletions(-)
> 
> $ diff -ur ./pre-patch/libavcodec/cbrt_tables.h ./powf-patch/libavcodec/cbrt_tables.h | diffstat
>  cbrt_tables.h | 2046 +++++++++++++++++++++++++++++-----------------------------
>  1 file changed, 1023 insertions(+), 1023 deletions(-)
> 
> $ diff -ur ./pre-patch/libavcodec/cbrt_tables.h ./pow-patch/libavcodec/cbrt_tables.h | diffstat
>  cbrt_tables.h | 1858 +++++++++++++++++++++++++++++-----------------------------
>  1 file changed, 929 insertions(+), 929 deletions(-)

thanks

ive no further comments, pick what you prefer

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Those who are too smart to engage in politics are punished by being
governed by those who are dumber. -- Plato 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130116/1749308d/attachment.asc>


More information about the ffmpeg-devel mailing list