[FFmpeg-devel] Patch for High color and High bit-depth support

James Zern jzern at google.com
Wed Apr 1 01:08:13 CEST 2015


On Mon, Mar 30, 2015 at 10:43 AM, Carl Eugen Hoyos <cehoyos at ag.or.at> wrote:
> Debargha Mukherjee <debargha <at> google.com> writes:
>
>> -   if (avctx->profile != FF_PROFILE_UNKNOWN)
>> -       enccfg.g_profile = avctx->profile;
>> +    if (avctx->profile != FF_PROFILE_UNKNOWN) {
>> +        enccfg.g_profile = avctx->profile;
>> +    }
>
> Please make this a separate patch, do not mix
> functional and cosmetic changes.
>
> +#ifdef VPX_IMG_FMT_HIGHBITDEPTH
> +    .pix_fmts       = (const enum AVPixelFormat[]){ AV_PIX_FMT_YUV420P,
> +                                                    AV_PIX_FMT_YUV422P,
> +                                                    AV_PIX_FMT_YUV444P,
>
> +#else
> +    .pix_fmts       = (const enum AVPixelFormat[]){ AV_PIX_FMT_YUV420P,
> +                                                    AV_PIX_FMT_YUV422P,
> +                                                    AV_PIX_FMT_YUV444P,
>
> I don't think this is more readable than having
> #ifdef VPX_IMG_FMT_HIGHBITDEPTH in the middle of
> the list. Do you disagree?
>

This is also modifying the pix_fmts for non-highbitdepth, that should
probably come first as a separate change.


More information about the ffmpeg-devel mailing list