[FFmpeg-devel] [PATCH] avcodec/aacdec_lpd: remove unused local variables

Michael Niedermayer michael at niedermayer.cc
Mon Jun 10 01:43:14 EEST 2024


On Sat, Jun 08, 2024 at 08:17:54PM -0400, Leo Izen wrote:
> int idx, and int first_tcx_flag are set but not used, so this commit
> removes their declarations and assignments.
> 
> Signed-off-by: Leo Izen <leo.izen at gmail.com>
> ---
>  libavcodec/aac/aacdec_lpd.c | 5 +----
>  1 file changed, 1 insertion(+), 4 deletions(-)
> 
> diff --git a/libavcodec/aac/aacdec_lpd.c b/libavcodec/aac/aacdec_lpd.c
> index 796edd2ab5..5e9909e361 100644
> --- a/libavcodec/aac/aacdec_lpd.c
> +++ b/libavcodec/aac/aacdec_lpd.c
> @@ -99,7 +99,7 @@ static void parse_qn(GetBitContext *gb, int *qn, int nk_mode, int no_qn)
>  static int parse_codebook_idx(GetBitContext *gb, uint32_t *kv,
>                                int nk_mode, int no_qn)
>  {
> -    int idx, n, nk;
> +    int n, nk;
>  
>      int qn[2];
>      parse_qn(gb, qn, nk_mode, no_qn);

libavcodec/aac/aacdec_lpd.c: In function ‘parse_codebook_idx’:
libavcodec/aac/aacdec_lpd.c:117:5: error: ‘idx’ undeclared (first use in this function)
  117 |     idx = get_bits(gb, 4*n);
      |     ^~~

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Many that live deserve death. And some that die deserve life. Can you give
it to them? Then do not be too eager to deal out death in judgement. For
even the very wise cannot see all ends. -- Gandalf
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20240610/96f51b0a/attachment.sig>


More information about the ffmpeg-devel mailing list