[FFmpeg-devel] [PATCH] configure option for 32-bit MPEG audio output
Wed Feb 11 12:46:02 CET 2009
On Wed, Feb 11, 2009 at 11:46:07AM +0100, Diego Biurrun wrote:
> On Tue, Feb 10, 2009 at 10:52:45PM -0800, Mike Melanson wrote:
> > I want to start doing automated testing of MPEG-1 audio conformance
> > vectors via FATE using the officially sanctioned RMS method. To that
> > end, FFmpeg needs to have configure-time support for 32-bit audio
> > output. The code is already in FFmpeg but it requires setting
> > CONFIG_AUDIO_NONSHORT manually in libavcodec/mpegaudio.h. I want to
> > remedy that with this patch.
> Looks fine, but..
> > --- libavcodec/mpegaudio.h (revision 17146)
> > +++ libavcodec/mpegaudio.h (working copy)
> > @@ -65,7 +63,7 @@
> > -#if CONFIG_MPEGAUDIO_HP && CONFIG_AUDIO_NONSHORT
> > +#if CONFIG_MPEGAUDIO_HP && CONFIG_AUDIO_LONG
> ..why not just
> if CONFIG_AUDIO_LONG
> Otherwise you have to pass two configure options to enable this..
is there any advantage of not enabling CONFIG_AUDIO_NONSHORT?
the option was added before full >16bit support in ffmpeg because
at that time non 16bit caused problems (like no support in ffmpeg.c)
It was supposed to be removed and set to 1 once 32bit was supported
of course doing a benchmark between both variants first wont hurt ...
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Dictatorship naturally arises out of democracy, and the most aggravated
form of tyranny and slavery out of the most extreme liberty. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: Digital signature
More information about the ffmpeg-devel