On Tue, 29 Nov 2016 22:10:47 +0100 Carl Eugen Hoyos <ceho...@ag.or.at> wrote:
> From add7f5d51491152af6d0431331543212c2c21ca4 Mon Sep 17 00:00:00 2001 > From: Carl Eugen Hoyos <ceho...@ag.or.at> > Date: Tue, 29 Nov 2016 22:09:21 +0100 > Subject: [PATCH] lavf/mov: Accept multiple fourcc for AVID 1:1. > > Fixes ticket #5982. > --- > libavformat/mov.c | 1 + > 1 file changed, 1 insertion(+) if (codec_tag && (codec_tag != format && + (codec_tag != AV_RL32("AV1x") || format != AV_RL32("AVup")) && // prores is allowed to have differing data format and codec tag codec_tag != AV_RL32("apcn") && codec_tag != AV_RL32("apch") && // so is dv (sigh) what about adding a comment like the prores/dv comments? (or in a separate patch, make a nicer comment for prores,avid and dv?) there are more than just these two AVID isoms. a quick search reveals... 'AV1X': 'Avid 1:1x (Quick Time)', 'AVD1': 'Avid DV (Quick Time)', 'AVDN': 'Avid DNxHD (Quick Time)', 'AVMP': 'Avid IMX (Quick Time)', 'AVUP': 'Avid 10bit Packed (Quick Time)', possibly AVin sample here : http://samples.ffmpeg.org/ffmpeg-bugs/trac/ticket524/AVCI50.mov do any of these isoms need to be added to the list in mov.c ? otherwise, looks ok if tested with our av1x/avup samples. thanks, -compn _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel