Re: [FFmpeg-devel] [PATCH v4 12/38] vaapi_encode_h26[45]: Make the AUD option a boolean

[myp...@gmail.com]

On Wed, Sep 19, 2018 at 6:33 AM Mark Thompson <s...@jkqxz.net> wrote:
>
> ---
>  libavcodec/vaapi_encode_h264.c | 2 +-
>  libavcodec/vaapi_encode_h265.c | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/libavcodec/vaapi_encode_h264.c b/libavcodec/vaapi_encode_h264.c
> index bba6848f03..0774ec25f9 100644
> --- a/libavcodec/vaapi_encode_h264.c
> +++ b/libavcodec/vaapi_encode_h264.c
> @@ -976,7 +976,7 @@ static const AVOption vaapi_encode_h264_options[] = {
>          { "ac",    NULL, 0, AV_OPT_TYPE_CONST, { .i64 = 1 }, INT_MIN, 
> INT_MAX, FLAGS, "coder" },
>
>      { "aud", "Include AUD",
> -      OFFSET(aud), AV_OPT_TYPE_INT, { .i64 = 0 }, 0, 1, FLAGS },
> +      OFFSET(aud), AV_OPT_TYPE_BOOL, { .i64 = 0 }, 0, 1, FLAGS },
>
>      { "sei", "Set SEI to include",
>        OFFSET(sei), AV_OPT_TYPE_FLAGS,
> diff --git a/libavcodec/vaapi_encode_h265.c b/libavcodec/vaapi_encode_h265.c
> index c40782a78e..6940a59240 100644
> --- a/libavcodec/vaapi_encode_h265.c
> +++ b/libavcodec/vaapi_encode_h265.c
> @@ -1093,7 +1093,7 @@ static const AVOption vaapi_encode_h265_options[] = {
>        OFFSET(qp), AV_OPT_TYPE_INT, { .i64 = 25 }, 0, 52, FLAGS },
>
>      { "aud", "Include AUD",
> -      OFFSET(aud), AV_OPT_TYPE_INT, { .i64 = 0 }, 0, 1, FLAGS },
> +      OFFSET(aud), AV_OPT_TYPE_BOOL, { .i64 = 0 }, 0, 1, FLAGS },
>
>      { "profile", "Set profile (general_profile_idc)",
>        OFFSET(profile), AV_OPT_TYPE_INT,
> --
LGTM
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel