On Sun, 2017-09-10 at 22:10 +0200, Michael Niedermayer wrote:
>  enum {
>      INDEX_MPEG2 = 0,
>      INDEX_AES3,
> @@ -159,6 +139,26 @@ enum {
>      INDEX_H264,
>  };
>  
> +static const struct {
> +    enum AVCodecID id;
> +    int index;
> +} mxf_essence_mappings[] = {
> +    { AV_CODEC_ID_MPEG2VIDEO, INDEX_MPEG2 },
> +    { AV_CODEC_ID_PCM_S24LE,  INDEX_AES3 },
> +    { AV_CODEC_ID_PCM_S16LE,  INDEX_AES3 },
> +    { AV_CODEC_ID_DVVIDEO,    INDEX_DV },
> +    { AV_CODEC_ID_DNXHD,      INDEX_DNXHD_1080p_10bit_HIGH },
> +    { AV_CODEC_ID_JPEG2000,   INDEX_JPEG2000 },
> +    { AV_CODEC_ID_H264,       INDEX_H264 },
> +    { AV_CODEC_ID_NONE }
> +};

This is tangentally relevant perhaps, but that INDEX_ enum should
really be type. Something like ULIndex and a comment with reference to
relevant spec section would be nice

/Tomas

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to