[FFmpeg-devel] [PATCH 3/3] avcodec/jpeglsdec: Avoid get_bits_long() where possible

Andreas Rheinhardt andreas.rheinhardt at outlook.com
Sat Jan 8 15:06:43 EET 2022


Andreas Rheinhardt:
> It is possible here, because the values of ff_log2_run used
> here are actually in the range 0..15 given that run_index is
> in the range 0..31.
> 
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>
> ---
>  libavcodec/jpeglsdec.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavcodec/jpeglsdec.c b/libavcodec/jpeglsdec.c
> index 32c0d2eb3f..269c71dc18 100644
> --- a/libavcodec/jpeglsdec.c
> +++ b/libavcodec/jpeglsdec.c
> @@ -277,7 +277,7 @@ static inline int ls_decode_line(JLSState *state, MJpegDecodeContext *s,
>              /* decode aborted run */
>              r = ff_log2_run[state->run_index[comp]];
>              if (r)
> -                r = get_bits_long(&s->gb, r);
> +                r = get_bits(&s->gb, r);
>              if (x + r * stride > w) {
>                  r = (w - x) / stride;
>              }
> 

Will apply this patch tonight unless there are objections.

- Andreas


More information about the ffmpeg-devel mailing list