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

[Paul B Mahol]

On 1/13/13, Nicolas George <nicolas.george at normalesup.org> wrote:
> 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.

Close as possible is what?

>
>
> 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
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>