On 15/04/2019 22:17, James Almer wrote:
> The spec defines it as an array of signed values, inferred to 0 when not
> present.
>
> Signed-off-by: James Almer <jamr...@gmail.com>
> ---
> Couldn't find any sample using it.
>
> libavcodec/cbs_h265.h | 2 +-
> libavcodec/cbs_h265_syntax_template.c | 7 +++++--
> 2 files changed, 6 insertions(+), 3 deletions(-)
>
> diff --git a/libavcodec/cbs_h265.h b/libavcodec/cbs_h265.h
> index d216caca2b..0c0e4f84b0 100644
> --- a/libavcodec/cbs_h265.h
> +++ b/libavcodec/cbs_h265.h
> @@ -659,7 +659,7 @@ typedef struct H265RawSEITimeCode {
> uint8_t minutes_flag[3];
> uint8_t hours_flag[3];
> uint8_t time_offset_length[3];
> - uint32_t time_offset_value[3];
> + int32_t time_offset_value[3];
> } H265RawSEITimeCode;
>
> typedef struct H265RawSEIMasteringDisplayColourVolume {
> diff --git a/libavcodec/cbs_h265_syntax_template.c
> b/libavcodec/cbs_h265_syntax_template.c
> index f1e1bb0e7e..bbd363a389 100644
> --- a/libavcodec/cbs_h265_syntax_template.c
> +++ b/libavcodec/cbs_h265_syntax_template.c
> @@ -1986,8 +1986,11 @@ static int FUNC(sei_time_code)(CodedBitstreamContext
> *ctx, RWContext *rw,
>
> us(5, time_offset_length[i], 0, 31, 1, i);
> if (current->time_offset_length[i] > 0)
> - us(current->time_offset_length[i], time_offset_value[i],
> - 0, MAX_UINT_BITS(current->time_offset_length[i]), 1, i);
> + is(current->time_offset_length[i], time_offset_value[i],
> + MIN_INT_BITS(current->time_offset_length[i]),
> + MAX_INT_BITS(current->time_offset_length[i]), 1, i);
> + else
> + infer(time_offset_value[i], 0);
> }
> }
>
>
LGTM.
Thanks,
- Mark
_______________________________________________
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".
