[FFmpeg-devel] [PATCH] avcodec/pcm: Increase a/mu-law encoding table sizes

2014-11-30 Thread Michael Niedermayer
This very slightly improves quality at the expense of 96kb more memory for tables Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/pcm.c |4 ++-- libavcodec/pcm_tablegen.h | 12 ++-- tests/ref/acodec/pcm-alaw |6 +++--- tests/ref/acodec/pcm-mulaw

Re: [FFmpeg-devel] [PATCH] avcodec/pcm: Increase a/mu-law encoding table sizes

2014-11-30 Thread Christophe Gisquet
Hi, 2014-11-30 12:11 GMT+01:00 Michael Niedermayer michae...@gmx.at: This very slightly improves quality at the expense of 96kb more memory for tables I'm not concerned by this case, but maybe put that under CONFIG_SMALL or something related? Also, out of curiosity rather than concern:

Re: [FFmpeg-devel] [PATCH] avcodec/pcm: Increase a/mu-law encoding table sizes

2014-11-30 Thread Michael Niedermayer
On Sun, Nov 30, 2014 at 12:19:50PM +0100, Christophe Gisquet wrote: Hi, 2014-11-30 12:11 GMT+01:00 Michael Niedermayer michae...@gmx.at: This very slightly improves quality at the expense of 96kb more memory for tables I'm not concerned by this case, but maybe put that under

Re: [FFmpeg-devel] [PATCH] avcodec/pcm: Increase a/mu-law encoding table sizes

2014-11-30 Thread Christophe Gisquet
2014-11-30 13:03 GMT+01:00 Michael Niedermayer michae...@gmx.at: not really, no, that was also why i posted a patch for this, i wasnt sure this is worth the extra table size No strong opinion here, I don't think the increased memory/potential speed impact are critical, in particular for this

Re: [FFmpeg-devel] [PATCH] avcodec/pcm: Increase a/mu-law encoding table sizes

2014-11-30 Thread Benoit Fouet
Hi, Le 30/11/2014 13:11, Christophe Gisquet a écrit : 2014-11-30 13:03 GMT+01:00 Michael Niedermayer michae...@gmx.at: not really, no, that was also why i posted a patch for this, i wasnt sure this is worth the extra table size No strong opinion here, I don't think the increased

Re: [FFmpeg-devel] [PATCH] avcodec/pcm: Increase a/mu-law encoding table sizes

2014-11-30 Thread Michael Niedermayer
On Sun, Nov 30, 2014 at 08:52:15PM +0100, Benoit Fouet wrote: Hi, Le 30/11/2014 13:11, Christophe Gisquet a écrit : 2014-11-30 13:03 GMT+01:00 Michael Niedermayer michae...@gmx.at: not really, no, that was also why i posted a patch for this, i wasnt sure this is worth the extra table size