[FFmpeg-devel] [PATCH] avcodec/cbs_av1: fix decoder/encoder_buffer_delay variable types

[Mark Thompson]

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