On Tue, Jul 14, 2020 at 7:08 PM Wang Cao <[email protected]> wrote:
>
> From: Wang Cao <[email protected]>
>
> Signed-off-by: Wang Cao <[email protected]>
> ---
> 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
[email protected]
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
[email protected] with subject "unsubscribe".