matroskaenc: don't duplicate title information
authorAnton Khirnov <wyskas@gmail.com>
Tue, 23 Nov 2010 08:24:41 +0000 (08:24 +0000)
committerAnton Khirnov <wyskas@gmail.com>
Tue, 23 Nov 2010 08:24:41 +0000 (08:24 +0000)
Title is already written into special fields designated for it --
per-segment Title, per-chapter ChapString and per-stream Name (yay for
consistent naming). Therefore, don't duplicate it in Tags.

Originally committed as revision 25813 to svn://svn.ffmpeg.org/ffmpeg/trunk

libavformat/matroskaenc.c

index 1430228..dc08c18 100644 (file)
@@ -32,6 +32,7 @@
 #include "libavutil/lfg.h"
 #include "libavcodec/xiph.h"
 #include "libavcodec/mpeg4audio.h"
+#include <strings.h>
 
 typedef struct ebml_master {
     int64_t         pos;                ///< absolute offset in the file where the master's elements start
@@ -742,7 +743,8 @@ static int mkv_write_tag(AVFormatContext *s, AVMetadata *m, unsigned int element
     end_ebml_master(s->pb, targets);
 
     while ((t = av_metadata_get(m, "", t, AV_METADATA_IGNORE_SUFFIX)))
-        mkv_write_simpletag(s->pb, t);
+        if (strcasecmp(t->key, "title"))
+            mkv_write_simpletag(s->pb, t);
 
     end_ebml_master(s->pb, tag);
     return 0;