[FFmpeg-devel] [PATCH]lavf/matroskadec: Set codec_tag also for audio codecs

Carl Eugen Hoyos cehoyos at ag.or.at
Thu Dec 10 23:50:45 CET 2015


Hi!

Attached patch is definitely a good idea imo, the mov demuxer also 
sets codec_tag reading the same atom, for "A_MS/ACM" codec_tag is 
already set.

Please comment, Carl Eugen
-------------- next part --------------
diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c
index aad567a..95cebdd 100644
--- a/libavformat/matroskadec.c
+++ b/libavformat/matroskadec.c
@@ -2124,6 +2124,7 @@ static int matroska_parse_tracks(AVFormatContext *s)
             }
         } else if (track->type == MATROSKA_TRACK_TYPE_AUDIO) {
             st->codec->codec_type  = AVMEDIA_TYPE_AUDIO;
+            st->codec->codec_tag   = fourcc;
             st->codec->sample_rate = track->audio.out_samplerate;
             st->codec->channels    = track->audio.channels;
             if (!st->codec->bits_per_coded_sample)


More information about the ffmpeg-devel mailing list