On 21/06/2024 23:20, Leo Izen wrote:
int idx, and int first_tcx_flag are set but not used, so this commit removes their declarations and assignments. --- libavcodec/aac/aacdec_lpd.c | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-)diff --git a/libavcodec/aac/aacdec_lpd.c b/libavcodec/aac/aacdec_lpd.c index 796edd2ab5..91a3268889 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); @@ -114,7 +114,7 @@ static int parse_codebook_idx(GetBitContext *gb, uint32_t *kv, } }- idx = get_bits(gb, 4*n);+ skip_bits(gb, 4*n);if (nk > 0)for (int i = 0; i < 8; i++) @@ -145,7 +145,6 @@ int ff_aac_ldp_parse_channel_stream(AACDecContext *ac, AACUSACConfig *usac, int k; const uint8_t *mod; int first_ldp_flag; - int first_tcx_flag;ce->ldp.acelp_core_mode = get_bits(gb, 3);ce->ldp.lpd_mode = get_bits(gb, 5); @@ -157,7 +156,6 @@ int ff_aac_ldp_parse_channel_stream(AACDecContext *ac, AACUSACConfig *usac, mod = ff_aac_lpd_mode_tab[ce->ldp.lpd_mode];first_ldp_flag = !ce->ldp.core_mode_last;- first_tcx_flag = 1; if (first_ldp_flag) ce->ldp.last_lpd_mode = -1; /* last_ldp_mode is a **STATEFUL** value */@@ -179,7 +177,6 @@ int ff_aac_ldp_parse_channel_stream(AACDecContext *ac, AACUSACConfig *usac,// parse_tcx_coding(); ce->ldp.last_lpd_mode = mod[k]; k += (1 << (mod[k] - 1)); - first_tcx_flag = 0; } }
Sorry, missed this patch, LGTM.
OpenPGP_0xA2FEA5F03F034464.asc
Description: OpenPGP public key
OpenPGP_signature.asc
Description: OpenPGP digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
