On Fri, May 20, 2016 at 10:11:46PM +0200, Alexandra Hájková wrote:
> --- a/libavcodec/golomb.h
> +++ b/libavcodec/golomb.h
> @@ -50,26 +50,24 @@ extern const uint8_t
> ff_interleaved_dirac_golomb_vlc_code[256];
> -static inline int get_ue_golomb(GetBitContext *gb)
> +static inline int get_ue_golomb(BitstreamContext *bb)
This should be bc, like everywhere else, same below.
> {
> unsigned int buf;
> + int ret;
>
> if (buf >= (1 << 27)) {
>
> - return ff_ue_golomb_vlc_code[buf];
> + ret = ff_ue_golomb_vlc_code[buf];
> + return ret;
> } else {
> int log = 2 * av_log2(buf) - 31;
> buf >>= log;
> buf--;
> - LAST_SKIP_BITS(re, gb, 32 - log);
> - CLOSE_READER(re, gb);
> + bitstream_skip(bb, 32 - log);
>
> return buf;
> }
The variables could have a smaller scope and somehow only one of them
matches the function return value.
Diego
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
