[FFmpeg-devel] [PATCH]lavf/mov: Accept multiple fourcc for AVID 1:1

compn tempn at mi.rr.com
Thu Dec 1 20:58:49 EET 2016


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 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


More information about the ffmpeg-devel mailing list