[FFmpeg-devel] [PATCH] lafv/matroska: add A_OPUS/EXPERIMENTAL codec name.

[Nicolas George]

The exact packing of Opus inside Matroska is not finalized.
Use A_OPUS/EXPERIMENTAL as codec name, like mkvtoolnix.
The A_OPUS name stays to let ffmpeg open files it has produced
until now, but newly produced file use the EXPERIMENTAL version.
Once the spec is stabilized it will be possible to consider
options to ensure compatibility with these files.

Signed-off-by: Nicolas George <nicolas.george at normalesup.org>
---
 libavformat/matroska.c |    1 +
 1 file changed, 1 insertion(+)


The file produced with this patch seems as close as possible to the output
of a fresh checkout of mkvtoolnix. We can not make plans on how to handle
files produced until now, not until the spec is stabilized. I suspect they
are very scarse anyway. At worst, ffmpeg with this patch is able to remux
them from A_OPUS to A_OPUS/EXPERIMENTAL.


diff --git a/libavformat/matroska.c b/libavformat/matroska.c
index 64d0a45..9b71846 100644
--- a/libavformat/matroska.c
+++ b/libavformat/matroska.c
@@ -32,6 +32,7 @@ const CodecTags ff_mkv_codec_tags[]={
     {"A_MPEG/L2"        , AV_CODEC_ID_MP2},
     {"A_MPEG/L1"        , AV_CODEC_ID_MP2},
     {"A_MPEG/L3"        , AV_CODEC_ID_MP3},
+    {"A_OPUS/EXPERIMENTAL",AV_CODEC_ID_OPUS},
     {"A_OPUS",            AV_CODEC_ID_OPUS},
     {"A_PCM/FLOAT/IEEE" , AV_CODEC_ID_PCM_F32LE},
     {"A_PCM/FLOAT/IEEE" , AV_CODEC_ID_PCM_F64LE},
-- 
1.7.10.4