On Tue, Jul 14, 2020 at 7:08 PM Wang Cao <doublee...@gmail.com> wrote:
>
> From: Wang Cao <doublee...@gmail.com>
>
> Signed-off-by: Wang Cao <wang...@google.com>
> ---
>  doc/encoders.texi      | 20 ++++++++++++++++++++
>  libavcodec/libaomenc.c | 32 ++++++++++++++++++++++++++++++++
>  libavcodec/version.h   |  2 +-
>  3 files changed, 53 insertions(+), 1 deletion(-)
>
> [...]
>      codecctl_int(avctx, AOME_SET_STATIC_THRESHOLD, ctx->static_thresh);
> @@ -1152,6 +1176,7 @@ static const AVOption options[] = {
>      { "psnr",            NULL,         0, AV_OPT_TYPE_CONST, {.i64 = 
> AOM_TUNE_PSNR}, 0, 0, VE, "tune"},
>      { "ssim",            NULL,         0, AV_OPT_TYPE_CONST, {.i64 = 
> AOM_TUNE_SSIM}, 0, 0, VE, "tune"},
>      FF_AV1_PROFILE_OPTS
> +#if AOM_ENCODER_ABI_VERSION >= 22

This wasn't done in the previous patch, so excluding these now would
create an incompatibility. Having them be a noop with older versions
is ok.

>      { "enable-rect-partitions", "Enable rectangular partitions", 
> OFFSET(enable_rect_partitions), AV_OPT_TYPE_BOOL, {.i64 = -1}, -1, 1, VE},
>      { "enable-1to4-partitions", "Enable 1:4/4:1 partitions",     
> OFFSET(enable_1to4_partitions), AV_OPT_TYPE_BOOL, {.i64 = -1}, -1, 1, VE},
>      { "enable-ab-partitions",   "Enable ab shape partitions",    
> OFFSET(enable_ab_partitions),   AV_OPT_TYPE_BOOL, {.i64 = -1}, -1, 1, VE},
> @@ -1162,6 +1187,13 @@ static const AVOption options[] = {
>      { "enable-smooth-intra",      "Enable smooth intra prediction mode",     
>            OFFSET(enable_smooth_intra),      AV_OPT_TYPE_BOOL, {.i64 = -1}, 
> -1, 1, VE},
>      { "enable-paeth-intra",       "Enable paeth predictor in intra 
> prediction",         OFFSET(enable_paeth_intra),       AV_OPT_TYPE_BOOL, 
> {.i64 = -1}, -1, 1, VE},
>      { "enable-palette",           "Enable palette prediction mode",          
>            OFFSET(enable_palette),           AV_OPT_TYPE_BOOL, {.i64 = -1}, 
> -1, 1, VE},
> +    { "enable-flip-idtx",          "Enable extended transform type",         
>               OFFSET(enable_flip_idtx),          AV_OPT_TYPE_BOOL, {.i64 = 
> -1}, -1, 1, VE},
> +    { "enable-tx64",               "Enable 64-pt transform",                 
>               OFFSET(enable_tx64),               AV_OPT_TYPE_BOOL, {.i64 = 
> -1}, -1, 1, VE},
> +    { "reduced-tx-type-set",       "Use reduced set of transform types. 
> Default is false", OFFSET(reduced_tx_type_set),       AV_OPT_TYPE_BOOL, {.i64 
> = -1}, -1, 1, VE},

We're taking the library default, so I think you can leave off the
default comment here like in the others.

> +    { "use-intra-dct-only",        "Use DCT only for INTRA modes",           
>               OFFSET(use_intra_dct_only),        AV_OPT_TYPE_BOOL, {.i64 = 
> -1}, -1, 1, VE},
> +    { "use-inter-dct-only",        "Use DCT only for INTER modes",           
>               OFFSET(use_inter_dct_only),        AV_OPT_TYPE_BOOL, {.i64 = 
> -1}, -1, 1, VE},
> +    { "use-intra-default-tx-only", "Use Default-transform only for INTRA 
> modes",           OFFSET(use_intra_default_tx_only), AV_OPT_TYPE_BOOL, {.i64 
> = -1}, -1, 1, VE},
> +#endif
>      { 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