Re: [FFmpeg-devel] [PATCH]lavf/matroska: Fix the A_MPEG/L1 codec_id

[Michael Niedermayer]

On Tue, Jan 03, 2017 at 03:46:06AM +0100, Carl Eugen Hoyos wrote:
> Hi!
> 
> Attached patch fixes MP1 muxing in mkv.
> 
> Please comment, Carl Eugen

>  matroska.c |2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 9ed063b5f9851ad17c82eb4896d9731e59b04d26  
> 0001-lavf-matroska-Fix-the-codec_id-for-mkv-tag-A_MPEG-L1.patch
> From 649977cf7af3734ec81204514874e9d16fcb6b29 Mon Sep 17 00:00:00 2001
> From: Carl Eugen Hoyos 
> Date: Tue, 3 Jan 2017 03:43:34 +0100
> Subject: [PATCH] lavf/matroska: Fix the codec_id for mkv tag A_MPEG/L1.
> 
> When the mapping was originally added AV_CODEC_ID_MP1 did not exist.
> ---
>  libavformat/matroska.c |2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

should be ok

thx

PS: please add a fate test

[...]

-- 
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


signature.asc
Description: Digital signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH]lavf/matroska: Fix the A_MPEG/L1 codec_id

[Carl Eugen Hoyos]

Hi!

Attached patch fixes MP1 muxing in mkv.

Please comment, Carl Eugen
From 649977cf7af3734ec81204514874e9d16fcb6b29 Mon Sep 17 00:00:00 2001
From: Carl Eugen Hoyos 
Date: Tue, 3 Jan 2017 03:43:34 +0100
Subject: [PATCH] lavf/matroska: Fix the codec_id for mkv tag A_MPEG/L1.

When the mapping was originally added AV_CODEC_ID_MP1 did not exist.
---
 libavformat/matroska.c |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavformat/matroska.c b/libavformat/matroska.c
index f3e1be7..c8e5341 100644
--- a/libavformat/matroska.c
+++ b/libavformat/matroska.c
@@ -35,7 +35,7 @@ const CodecTags ff_mkv_codec_tags[]={
 {"A_FLAC"   , AV_CODEC_ID_FLAC},
 {"A_MLP", AV_CODEC_ID_MLP},
 {"A_MPEG/L2", AV_CODEC_ID_MP2},
-{"A_MPEG/L1", AV_CODEC_ID_MP2},
+{"A_MPEG/L1", AV_CODEC_ID_MP1},
 {"A_MPEG/L3", AV_CODEC_ID_MP3},
 {"A_OPUS"   , AV_CODEC_ID_OPUS},
 {"A_OPUS/EXPERIMENTAL",AV_CODEC_ID_OPUS},
-- 
1.7.10.4

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel