[FFmpeg-devel] [PATCH] avcodec/get_bits: Document skip_bits_long()

Thilo Borgmann thilo.borgmann at mail.de
Fri Mar 23 23:02:39 EET 2018


Am 23.03.18 um 20:20 schrieb Michael Niedermayer:
> Found-by: Kieran
> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> ---
>  libavcodec/get_bits.h | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/libavcodec/get_bits.h b/libavcodec/get_bits.h
> index 0c7f5ff0c6..3ec45e7ab6 100644
> --- a/libavcodec/get_bits.h
> +++ b/libavcodec/get_bits.h
> @@ -201,6 +201,13 @@ static inline int get_bits_count(const GetBitContext *s)
>      return s->index;
>  }
>  
> +/**
> + * Skips the specified number of bits.
> + * @param n the number of bits to skip,
> + *          For the UNCHECKED_BITSTREAM_READER this must not cause the distance
> + *          from the start to overflow int32_t. Staying within the bitstream + padding
> + *          is sufficient too.
                  ^^^^^^^^^^
Shouldn't this be "required" or "necessary"?
And nit: "something, too."


-Thilo



More information about the ffmpeg-devel mailing list