[FFmpeg-devel] [PATCH] ffmpeg_opt: deprecate the hwaccel_lax_profile_check option

Mark Thompson sw at jkqxz.net
Sun Oct 15 23:22:54 EEST 2017


On 13/10/17 08:07, Jun Zhao wrote:
> From 40c45849b5146688ce6f88ca4fe20b771fb2de46 Mon Sep 17 00:00:00 2001
> From: Jun Zhao <jun.zhao at intel.com>
> Date: Fri, 13 Oct 2017 02:53:01 -0400
> Subject: [PATCH] ffmpeg_opt: deprecate the hwaccel_lax_profile_check option
> 
> deprecate hwaccel_lax_profile_check. This only was used for VAAPI
> hwaccel decoder, now use per-stream hwaccel_flags for all hwaccel
> decoders.
> 
> Signed-off-by: Jun Zhao <jun.zhao at intel.com>
> ---
>  fftools/ffmpeg_opt.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/fftools/ffmpeg_opt.c b/fftools/ffmpeg_opt.c
> index 100fa76e46..f1c96ce08c 100644
> --- a/fftools/ffmpeg_opt.c
> +++ b/fftools/ffmpeg_opt.c
> @@ -100,7 +100,7 @@ const HWAccel hwaccels[] = {
>  #endif
>      { 0 },
>  };
> -int hwaccel_lax_profile_check = 0;
> +attribute_deprecated int hwaccel_lax_profile_check = 0;
>  AVBufferRef *hw_device_ctx;
>  HWDevice *filter_hw_device;
>  
> -- 
> 2.11.0

Having thought about this a bit further and searched for uses, I now agree with your original position that the option should just be removed.  Results in google are /all/ patches to do with it, links to the code, or appearances of the message which was in some versions last year suggesting its use.

I'll apply your original patch for this (<https://lists.ffmpeg.org/pipermail/ffmpeg-devel/2017-October/217670.html>) tomorrow if there are no objections.

Thanks,

- Mark

(Aside: the patch just sent doesn't actually do what you want - it only deprecates the variable and triggers warnings building ffmpeg.c, but doesn't say anything about the option itself at runtime.)


More information about the ffmpeg-devel mailing list