avformat/movenc: add reason for minf->hdlr in MOV only
authorGyan Doshi <ffmpeg@gyani.pro>
Sat, 14 Jul 2018 06:31:58 +0000 (12:01 +0530)
committerGyan Doshi <ffmpeg@gyani.pro>
Sat, 14 Jul 2018 06:33:50 +0000 (12:03 +0530)
libavformat/movenc.c

index 804deef..54a248c 100644 (file)
@@ -2674,7 +2674,7 @@ static int mov_write_minf_tag(AVFormatContext *s, AVIOContext *pb, MOVMuxContext
     } else if (track->tag == MKTAG('g','p','m','d')) {
         mov_write_gmhd_tag(pb, track);
     }
-    if (track->mode == MODE_MOV) /* FIXME: Why do it for MODE_MOV only ? */
+    if (track->mode == MODE_MOV) /* ISO 14496-12 8.4.3.1 specifies hdlr only within mdia or meta boxes */
         mov_write_hdlr_tag(s, pb, NULL);
     mov_write_dinf_tag(pb);
     if ((ret = mov_write_stbl_tag(s, pb, mov, track)) < 0)