On 10/26/2020 6:21 PM, Bohan Li wrote:

> Regarding the last comment, the enable-keyframe-filtering parameter was a
> boolean, but recently there is one more option added to libaom
> (--enable-keyframe-filtering=2), so I thought it would be better to use
>  AV_OPT_TYPE_INT here so people who build libaom from source could use that
> option.
> 
> Please let me know if there are any concerns with it.

No, if it was extended then it's correct as INT.

> Add the option to use -enable-keyframe-filtering with libaom-av1
> codec.  The option controls the encoder behavior on performing
> temporal filtering on keyframes.

By missing documentation i meant an entry in doc/encoder.texi
Look for the libaom-av1 section and add the new option at the end.

This blurb in the commit message is ok and can stay, too.

> 
> Signed-off-by: Bohan Li <boha...@google.com>
> ---
>  libavcodec/libaomenc.c | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/libavcodec/libaomenc.c b/libavcodec/libaomenc.c
> index 2b0581b15a..77c25770a4 100644
> --- a/libavcodec/libaomenc.c
> +++ b/libavcodec/libaomenc.c
> @@ -124,6 +124,7 @@ typedef struct AOMEncoderContext {
>      int enable_diff_wtd_comp;
>      int enable_dist_wtd_comp;
>      int enable_dual_filter;
> +    int enable_keyframe_filtering;
>  } AOMContext;
>  
>  static const char *const ctlidstr[] = {
> @@ -192,6 +193,7 @@ static const char *const ctlidstr[] = {
>      [AV1E_SET_REDUCED_REFERENCE_SET]    = "AV1E_SET_REDUCED_REFERENCE_SET",
>      [AV1E_SET_ENABLE_SMOOTH_INTERINTRA] = 
> "AV1E_SET_ENABLE_SMOOTH_INTERINTRA",
>      [AV1E_SET_ENABLE_REF_FRAME_MVS]     = "AV1E_SET_ENABLE_REF_FRAME_MVS",
> +    [AV1E_SET_ENABLE_KEYFRAME_FILTERING] = 
> "AV1E_SET_ENABLE_KEYFRAME_FILTERING"
>  #endif
>  };
>  
> @@ -812,6 +814,8 @@ static av_cold int aom_init(AVCodecContext *avctx,
>          codecctl_int(avctx, AV1E_SET_ENABLE_ONESIDED_COMP, 
> ctx->enable_onesided_comp);
>      if (ctx->enable_smooth_interintra >= 0)
>          codecctl_int(avctx, AV1E_SET_ENABLE_SMOOTH_INTERINTRA, 
> ctx->enable_smooth_interintra);
> +    if (ctx->enable_keyframe_filtering >= 0)
> +        codecctl_int(avctx, AV1E_SET_ENABLE_KEYFRAME_FILTERING, 
> ctx->enable_keyframe_filtering);
>  #endif
>  
>      codecctl_int(avctx, AOME_SET_STATIC_THRESHOLD, ctx->static_thresh);
> @@ -1261,6 +1265,7 @@ static const AVOption options[] = {
>      { "enable-masked-comp",           "Enable masked compound",              
>               OFFSET(enable_masked_comp),           AV_OPT_TYPE_BOOL, {.i64 = 
> -1}, -1, 1, VE},
>      { "enable-interintra-comp",       "Enable interintra compound",          
>               OFFSET(enable_interintra_comp),       AV_OPT_TYPE_BOOL, {.i64 = 
> -1}, -1, 1, VE},
>      { "enable-smooth-interintra",     "Enable smooth interintra mode",       
>               OFFSET(enable_smooth_interintra),     AV_OPT_TYPE_BOOL, {.i64 = 
> -1}, -1, 1, VE},
> +    { "enable-keyframe-filtering",    "Keyframe filtering type",             
>               OFFSET(enable_keyframe_filtering),    AV_OPT_TYPE_INT,  {.i64 = 
> -1}, -1, 3, VE},
>      { NULL },
>  };
>  
> 

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to