[FFmpeg-devel] [PATCH 2/2] avcodec/libx264: fix compilation with x264 builds >= 153

[Ricardo Constantino]

On 26 December 2017 at 00:44, James Almer <jamrial at gmail.com> wrote:

> x264 now supports multibitdepth builds, with a slightly changed API to
> request bitdepth during initialization.
>
> Signed-off-by: James Almer <jamrial at gmail.com>
> ---
> libx264 can still be compiled with only 8 or 10 bit format support even
> after the multibitdepth change, so take that into consideration.
>
>  libavcodec/libx264.c | 30 +++++++++++++++++++++++++++---
>  1 file changed, 27 insertions(+), 3 deletions(-)
>
> diff --git a/libavcodec/libx264.c b/libavcodec/libx264.c
> index 2d36c5e566..20ed59fbc4 100644
> --- a/libavcodec/libx264.c
> +++ b/libavcodec/libx264.c
> @@ -454,6 +454,7 @@ static av_cold int X264_init(AVCodecContext *avctx)
>  {
>      X264Context *x4 = avctx->priv_data;
>      AVCPBProperties *cpb_props;
> +    const av_unused AVPixFmtDescriptor *desc = av_pix_fmt_desc_get(avctx->
> pix_fmt);
>      int sw,sh;
>
>      if (avctx->global_quality > 0)
> @@ -724,6 +725,9 @@ FF_ENABLE_DEPRECATION_WARNINGS
>
>      x4->params.i_width          = avctx->width;
>      x4->params.i_height         = avctx->height;
> +#if X264_BUILD >= 153
> +    x4->params.i_bitdepth       = desc->comp[0].depth;
> +#endif
>      av_reduce(&sw, &sh, avctx->sample_aspect_ratio.num,
> avctx->sample_aspect_ratio.den, 4096);
>      x4->params.vui.i_sar_width  = sw;
>      x4->params.vui.i_sar_height = sh;
> @@ -837,6 +841,24 @@ FF_ENABLE_DEPRECATION_WARNINGS
>      return 0;
>  }
>
> +static const enum AVPixelFormat pix_fmts[] = {
> +    AV_PIX_FMT_YUV420P,
> +    AV_PIX_FMT_YUVJ420P,
> +    AV_PIX_FMT_YUV422P,
> +    AV_PIX_FMT_YUVJ422P,
> +    AV_PIX_FMT_YUV444P,
> +    AV_PIX_FMT_YUVJ444P,
> +    AV_PIX_FMT_YUV420P10,
> +    AV_PIX_FMT_YUV422P10,
> +    AV_PIX_FMT_YUV444P10,
> +    AV_PIX_FMT_NV12,
> +    AV_PIX_FMT_NV16,
> +    AV_PIX_FMT_NV20,
> +#ifdef X264_CSP_NV21
> +    AV_PIX_FMT_NV21,
> +#endif
> +    AV_PIX_FMT_NONE
> +};
>  static const enum AVPixelFormat pix_fmts_8bit[] = {
>      AV_PIX_FMT_YUV420P,
>      AV_PIX_FMT_YUVJ420P,
> @@ -874,12 +896,14 @@ static const enum AVPixelFormat pix_fmts_8bit_rgb[]
> = {
>
>  static av_cold void X264_init_static(AVCodec *codec)
>  {
> -    if (x264_bit_depth == 8)
> +    if (X264_BIT_DEPTH == 8)
>          codec->pix_fmts = pix_fmts_8bit;
> -    else if (x264_bit_depth == 9)
> +    else if (X264_BIT_DEPTH == 9)
>          codec->pix_fmts = pix_fmts_9bit;
> -    else if (x264_bit_depth == 10)
> +    else if (X264_BIT_DEPTH == 10)
>          codec->pix_fmts = pix_fmts_10bit;
> +    else /* X264_BIT_DEPTH == 0 */
> +        codec->pix_fmts = pix_fmts;
>  }
>
>  #define OFFSET(x) offsetof(X264Context, x)
> --
> 2.15.0
>
>
LGTM


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