2011/7/30 Alex Converse <[email protected]> > On Fri, Jul 29, 2011 at 11:10 AM, Yusuke Nakamura > <[email protected]> wrote: > > 2011/7/29 Diego Biurrun <[email protected]> > >> > >> --- > >> 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 > >> [email protected] > >> 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 > [email protected] > https://lists.libav.org/mailman/listinfo/libav-devel >
Those brands are ISO Base Media's, not MP4's. Where are they described? I cannot find any description of iods in 14496-12 except for AnnexE of the 3rd Edition. Assuming your opinion is correct, why, for instance, doesn't male_amr122.3gp in 14496-4 2004 Amd24 have iods box? If a box in a compatible brand is mandatory, that box shall be in the file, and in absence of that box the file is not compatible.
_______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
