Re: [FFmpeg-devel] [PATCH] mpegaudiodec: share identical tables between float and fixed decoder.

2014-09-01 Thread Reimar Döffinger
On Mon, Sep 01, 2014 at 11:50:18AM +0200, Michael Niedermayer wrote: > On Mon, Sep 01, 2014 at 09:30:04AM +0200, Reimar Döffinger wrote: > > On 01.09.2014, at 02:01, Michael Niedermayer wrote: > > > On Sun, Aug 31, 2014 at 10:47:09PM +0200, Reimar Döffinger wrote: > > >> ff_mpa_table_4_3_value alo

Re: [FFmpeg-devel] [PATCH] mpegaudiodec: share identical tables between float and fixed decoder.

2014-09-01 Thread Michael Niedermayer
On Mon, Sep 01, 2014 at 09:30:04AM +0200, Reimar Döffinger wrote: > On 01.09.2014, at 02:01, Michael Niedermayer wrote: > > On Sun, Aug 31, 2014 at 10:47:09PM +0200, Reimar Döffinger wrote: > >> ff_mpa_table_4_3_value alone saves over 128kB. > > > > i wonder if that table shouldnt be something di

Re: [FFmpeg-devel] [PATCH] mpegaudiodec: share identical tables between float and fixed decoder.

2014-09-01 Thread Reimar Döffinger
On 01.09.2014, at 02:01, Michael Niedermayer wrote: > On Sun, Aug 31, 2014 at 10:47:09PM +0200, Reimar Döffinger wrote: >> ff_mpa_table_4_3_value alone saves over 128kB. > > i wonder if that table shouldnt be something different in the float > implementation. > Also i think anyone caring about fi

Re: [FFmpeg-devel] [PATCH] mpegaudiodec: share identical tables between float and fixed decoder.

2014-08-31 Thread Michael Niedermayer
On Sun, Aug 31, 2014 at 10:47:09PM +0200, Reimar Döffinger wrote: > ff_mpa_table_4_3_value alone saves over 128kB. i wonder if that table shouldnt be something different in the float implementation. Also i think anyone caring about file size would likely not build the float and fixed point mp3 dec

[FFmpeg-devel] [PATCH] mpegaudiodec: share identical tables between float and fixed decoder.

2014-08-31 Thread Reimar Döffinger
ff_mpa_table_4_3_value alone saves over 128kB. Signed-off-by: Reimar Döffinger --- libavcodec/Makefile| 24 +++--- libavcodec/mpegaudio_tablegen.c| 13 +-- libavcodec/mpegaudio_tablegen.h| 26 +++--- libavcodec/mpegaudiodec_common.c | 135