[FFmpeg-devel] [PATCH 1/3] api/api-band-test: Remove write-only variable

Andreas Rheinhardt andreas.rheinhardt at outlook.com
Tue Jun 11 13:23:37 EEST 2024


Andreas Rheinhardt:
> Besides being write only it had the wrong type:
> An uint8_t is definitely not enough to store the size
> of these buffers.
> 
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>
> ---
>  tests/api/api-band-test.c | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/tests/api/api-band-test.c b/tests/api/api-band-test.c
> index bbaa6edb58..d40e31b850 100644
> --- a/tests/api/api-band-test.c
> +++ b/tests/api/api-band-test.c
> @@ -31,7 +31,6 @@
>  #include "libavutil/imgutils.h"
>  
>  uint8_t *slice_byte_buffer;
> -uint8_t slice_byte_buffer_size;
>  int draw_horiz_band_called;
>  
>  static void draw_horiz_band(AVCodecContext *ctx, const AVFrame *fr, int offset[4],
> @@ -158,7 +157,6 @@ static int video_decode(const char *input_filename)
>          return AVERROR(ENOMEM);
>      }
>      memset(slice_byte_buffer, 0, byte_buffer_size);
> -    slice_byte_buffer_size = byte_buffer_size;
>  
>      result = 0;
>      while (result >= 0) {

Will apply this patchset tomorrow unless there are objections.

- Andreas



More information about the ffmpeg-devel mailing list