[FFmpeg-devel] [PATCH] avcodec/cbs_av1: fix decoder/encoder_buffer_delay variable types
Mark Thompson
sw at jkqxz.net
Mon Nov 5 02:32:47 EET 2018
On 04/11/18 20:13, James Almer wrote:
> buffer_delay_length_minus_1 is five bits long, meaning decode_buffer_delay and
> encoder_buffer_delay can have values up to 32 bits long.
>
> Signed-off-by: James Almer <jamrial at gmail.com>
> ---
> See https://0x0.st/sIv-.ivf
>
> libavcodec/cbs_av1.h | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/libavcodec/cbs_av1.h b/libavcodec/cbs_av1.h
> index b66a09c389..614a0bf108 100644
> --- a/libavcodec/cbs_av1.h
> +++ b/libavcodec/cbs_av1.h
> @@ -87,8 +87,8 @@ typedef struct AV1RawSequenceHeader {
> uint8_t seq_level_idx[AV1_MAX_OPERATING_POINTS];
> uint8_t seq_tier[AV1_MAX_OPERATING_POINTS];
> uint8_t decoder_model_present_for_this_op[AV1_MAX_OPERATING_POINTS];
> - uint8_t decoder_buffer_delay[AV1_MAX_OPERATING_POINTS];
> - uint8_t encoder_buffer_delay[AV1_MAX_OPERATING_POINTS];
> + uint32_t decoder_buffer_delay[AV1_MAX_OPERATING_POINTS];
> + uint32_t encoder_buffer_delay[AV1_MAX_OPERATING_POINTS];
> uint8_t low_delay_mode_flag[AV1_MAX_OPERATING_POINTS];
> uint8_t initial_display_delay_present_for_this_op[AV1_MAX_OPERATING_POINTS];
> uint8_t initial_display_delay_minus_1[AV1_MAX_OPERATING_POINTS];
>
Yep; LGTM.
Thanks,
- Mark
More information about the ffmpeg-devel
mailing list