On Do, 2023-08-03 at 14:01 +0800, fei.w.wang-at-intel@ffmpeg.org wrote:
> From: Fei Wang
>
> Signed-off-by: Fei Wang
> ---
> libavcodec/av1.h | 7 +++
> libavcodec/cbs_av1_syntax_template.c | 4 ++--
> 2 files changed, 9 insertions(+), 2 deletions(-)
>
> diff --git a/libavcodec/av1.h b/libavcodec/av1.h
> index 384f7cddc7..8704bc41c1 100644
> --- a/libavcodec/av1.h
> +++ b/libavcodec/av1.h
> @@ -175,6 +175,13 @@ enum {
> AV1_RESTORE_SWITCHABLE = 3,
> };
>
> +// TX mode (section 6.8.21)
> +enum {
> + AV1_ONLY_4X4 = 0,
> + AV1_TX_MODE_LARGEST = 1,
> + AV1_TX_MODE_SELECT = 2,
> +};
> +
> // Sequence Headers are actually unbounded because one can use
> // an arbitrary number of leading zeroes when encoding via uvlc.
> // The following estimate is based around using the lowest number
> diff --git a/libavcodec/cbs_av1_syntax_template.c
> b/libavcodec/cbs_av1_syntax_template.c
> index a747e17784..3a5cafbfb7 100644
> --- a/libavcodec/cbs_av1_syntax_template.c
> +++ b/libavcodec/cbs_av1_syntax_template.c
> @@ -1028,9 +1028,9 @@ static int FUNC(read_tx_mode)(CodedBitstreamContext
> *ctx, RWContext *rw,
> int err;
>
> if (priv->coded_lossless)
> - infer(tx_mode, 0);
> + infer(tx_mode, AV1_ONLY_4X4);
> else
> - increment(tx_mode, 1, 2);
> + increment(tx_mode, AV1_TX_MODE_LARGEST, AV1_TX_MODE_SELECT);
>
> return 0;
> }
This patchset looks good and works well for me. I'll push it in a few days if
there are no comments.
Thanks
Haihao
___
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".
