On Fri, Jul 29, 2011 at 11:10 AM, Yusuke Nakamura <muken.the.vfrman...@gmail.com> wrote: > 2011/7/29 Diego Biurrun <di...@biurrun.de> >> >> --- >> libavformat/movenc.c | 17 ----------------- >> 1 files changed, 0 insertions(+), 17 deletions(-) >> >> diff --git a/libavformat/movenc.c b/libavformat/movenc.c >> index c5750d1..10d7abb 100644 >> --- a/libavformat/movenc.c >> +++ b/libavformat/movenc.c >> @@ -1400,22 +1400,6 @@ static int mov_write_trak_tag(AVIOContext *pb, >> MOVTrack *track, AVStream *st) >> return updateSize(pb, pos); >> } >> >> -#if 0 >> -/* TODO: Not sorted out, but not necessary either */ >> -static int mov_write_iods_tag(AVIOContext *pb, MOVMuxContext *mov) >> -{ >> - avio_wb32(pb, 0x15); /* size */ >> - ffio_wfourcc(pb, "iods"); >> - avio_wb32(pb, 0); /* version & flags */ >> - avio_wb16(pb, 0x1007); >> - avio_w8(pb, 0); >> - avio_wb16(pb, 0x4fff); >> - avio_wb16(pb, 0xfffe); >> - avio_wb16(pb, 0x01ff); >> - return 0x15; >> -} >> -#endif >> - >> static int mov_write_mvhd_tag(AVIOContext *pb, MOVMuxContext *mov) >> { >> int maxTrackID = 1, i; >> @@ -1821,7 +1805,6 @@ static int mov_write_moov_tag(AVIOContext *pb, >> MOVMuxContext *mov, >> } >> >> mov_write_mvhd_tag(pb, mov); >> - //mov_write_iods_tag(pb, mov); >> for (i=0; i<mov->nb_streams; i++) { >> if(mov->tracks[i].entry > 0) { >> mov_write_trak_tag(pb, &(mov->tracks[i]), i < s->nb_streams ? >> s->streams[i] : NULL); >> -- >> 1.7.1 >> >> _______________________________________________ >> libav-devel mailing list >> libav-devel@libav.org >> https://lists.libav.org/mailman/listinfo/libav-devel > > Oh? > libav doesn't support MP4 ver.2 (mp42)? > Then, this is spec violation. > For MP4 ver.1 (mp41), Object Descriptor Box (iods) is mandatory. > For MP4 ver.2, not mandatory. >
It is also mandatory in the 'isom', 'avc1' ,'iso2', and 'iso3' brands _______________________________________________ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel