On Tue, Jun 06, 2017 at 06:51:09PM +0200, wm4 wrote:
> --- a/libavcodec/dxva2.c
> +++ b/libavcodec/dxva2.c
> @@ -301,3 +301,12 @@ end:
> +
> +int ff_dxva2_is_d3d11(const AVCodecContext *avctx)
> +{
> +#if CONFIG_D3D11VA
> +    return avctx->pix_fmt == AV_PIX_FMT_D3D11VA_VLD;
> +#else
> +    return 0;
> +#endif
> +}

No need for ifdefs:

  int ff_dxva2_is_d3d11(const AVCodecContext *avctx)
  {
      return CONFIG_D3D11VA ? avctx->pix_fmt == AV_PIX_FMT_D3D11VA_VLD : 0;
  }

or

  int ff_dxva2_is_d3d11(const AVCodecContext *avctx)
  {
      if (CONFIG_D3D11VA)
          return avctx->pix_fmt == AV_PIX_FMT_D3D11VA_VLD;
      else
          return 0;
  }

> --- a/libavcodec/dxva2_internal.h
> +++ b/libavcodec/dxva2_internal.h
> @@ -59,21 +59,23 @@ typedef union {
>  #endif
>  } AVDXVAContext;
>  
> +#define DXVA_CONTEXT(avctx) ((AVDXVAContext*)(avctx)->hwaccel_context)

nit: (AVDXVAContext *)

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to