[FFmpeg-devel] [PATCH]lavf/mov: Accept multiple fourcc for AVID 1:1
ffmpeg at 2foo.be
Tue Dec 6 09:49:14 EET 2016
On 2016-12-01 19:58, compn wrote:
> On Tue, 29 Nov 2016 22:10:47 +0100
> Carl Eugen Hoyos <cehoyos at ag.or.at> wrote:
>> From add7f5d51491152af6d0431331543212c2c21ca4 Mon Sep 17 00:00:00 2001
>> From: Carl Eugen Hoyos <cehoyos at 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
> 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 :
> 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.
Can this patch be pushed to master please? Seems to work for me (I
orignally reported the issue). If any work still needs to be done , tell
me then I can have a look if I'm capable of doing it.
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
More information about the ffmpeg-devel