On Mo, Jan 02, 2012 at 17:50:56 (CET), Reinhard Tartler wrote:

[...]

> --- a/libavcodec/avcodec.h
> +++ b/libavcodec/avcodec.h
> @@ -208,6 +208,7 @@ enum CodecID {
>      CODEC_ID_8SVX_FIB,
>  #endif
>      CODEC_ID_ESCAPE124,
> +    CODEC_ID_ESCAPE130,
>      CODEC_ID_DIRAC,
>      CODEC_ID_BFI,
>      CODEC_ID_CMV,

[...]

I have been pointed out by Nicolas that CODEC_IDs may only be added at
the end of the CodecID enum to not break ABI compatibility. The comment
at the top of the struct confirms that. Therefore, I have added the
following patch to my working branch. This patch will be squashed in the
final commit, of course.

diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h
index 01af3c2..06bb608 100644
--- a/libavcodec/avcodec.h
+++ b/libavcodec/avcodec.h
@@ -208,7 +208,6 @@ enum CodecID {
     CODEC_ID_8SVX_FIB,
 #endif
     CODEC_ID_ESCAPE124,
-    CODEC_ID_ESCAPE130,
     CODEC_ID_DIRAC,
     CODEC_ID_BFI,
     CODEC_ID_CMV,
@@ -255,6 +254,7 @@ enum CodecID {
     CODEC_ID_VBLE,
     CODEC_ID_DXTORY,
     CODEC_ID_V410,
+    CODEC_ID_ESCAPE130,
 
     /* various PCM "codecs" */
     CODEC_ID_FIRST_AUDIO = 0x10000,     ///< A dummy id pointing at the start 
of audio codecs


-- 
Gruesse/greetings,
Reinhard Tartler, KeyID 945348A4
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to