[FFmpeg-cvslog] r10889 - trunk/libavcodec/allcodecs.c

Aurelien Jacobs aurel
Wed Oct 31 23:55:46 CET 2007


michael wrote:

> Author: michael
> Date: Wed Oct 31 23:39:10 2007
> New Revision: 10889
> 
> Log:
> prefer our native vorbis decoder over libvorbis

It also affect encoder...
And libvorbis encoder is certainly preferred over the native one.

> Modified:
>    trunk/libavcodec/allcodecs.c
> 
> Modified: trunk/libavcodec/allcodecs.c
> ==============================================================================
> --- trunk/libavcodec/allcodecs.c	(original)
> +++ trunk/libavcodec/allcodecs.c	Wed Oct 31 23:39:10 2007
> @@ -191,7 +191,6 @@ void avcodec_register_all(void)
>      REGISTER_ENCDEC  (LIBGSM_MS, libgsm_ms);
>      REGISTER_ENCODER (LIBMP3LAME, libmp3lame);
>      REGISTER_ENCODER (LIBTHEORA, libtheora);
> -    REGISTER_ENCDEC  (LIBVORBIS, libvorbis);
>      REGISTER_DECODER (MACE3, mace3);
>      REGISTER_DECODER (MACE6, mace6);
>      REGISTER_ENCDEC  (MP2, mp2);
> @@ -215,6 +214,7 @@ void avcodec_register_all(void)
>      REGISTER_ENCDEC  (WMAV1, wmav1);
>      REGISTER_ENCDEC  (WMAV2, wmav2);
>      REGISTER_DECODER (WS_SND1, ws_snd1);
> +    REGISTER_ENCDEC  (LIBVORBIS, libvorbis);

I don't like this change very much...
Default ./configure will enable native vorbis and disable libvorbis.
If user intentionally enable libvorbis, that's probably because he
want to use it. Thus it should probably be preferred over vorbis.

Oh, and the best way to fix this anyway is to drop support for
libvorbis decoding ;-)

Aurel




More information about the ffmpeg-cvslog mailing list