[FFmpeg-devel] [PATCH 2/2] avcodec/roqvideodec: Remove set-but-unused frame stats

Paul B Mahol onemda at gmail.com
Sun Aug 30 18:25:14 EEST 2020


On 8/30/20, Andreas Rheinhardt <andreas.rheinhardt at gmail.com> wrote:
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
> ---
> Stange that compilers didn't warn about this. Probably because it is an
> array.
>

probably ok

>  libavcodec/roqvideodec.c | 3 ---
>  1 file changed, 3 deletions(-)
>
> diff --git a/libavcodec/roqvideodec.c b/libavcodec/roqvideodec.c
> index 93c6e9ddf3..36b4ddf755 100644
> --- a/libavcodec/roqvideodec.c
> +++ b/libavcodec/roqvideodec.c
> @@ -39,7 +39,6 @@ static void roqvideo_decode_frame(RoqContext *ri,
> GetByteContext *gb)
>      unsigned long chunk_size = 0;
>      int i, j, k, nv1, nv2, vqflg = 0, vqflg_pos = -1;
>      int vqid, xpos, ypos, xp, yp, x, y, mx, my;
> -    int frame_stats[2][4] = {{0},{0}};
>      roq_qcell *qcell;
>      int64_t chunk_start;
>
> @@ -89,7 +88,6 @@ static void roqvideo_decode_frame(RoqContext *ri,
> GetByteContext *gb)
>                      vqflg_pos = 7;
>                  }
>                  vqid = (vqflg >> (vqflg_pos * 2)) & 0x3;
> -                frame_stats[0][vqid]++;
>                  vqflg_pos--;
>
>                  switch(vqid) {
> @@ -124,7 +122,6 @@ static void roqvideo_decode_frame(RoqContext *ri,
> GetByteContext *gb)
>                              vqflg_pos = 7;
>                          }
>                          vqid = (vqflg >> (vqflg_pos * 2)) & 0x3;
> -                        frame_stats[1][vqid]++;
>                          vqflg_pos--;
>                          switch(vqid) {
>                          case RoQ_ID_MOT:
> --
> 2.20.1
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".


More information about the ffmpeg-devel mailing list