diff options
| author | Yotam Ofek <yotam.ofek@gmail.com> | 2024-04-28 09:37:11 +0000 |
|---|---|---|
| committer | Lynne <dev@lynne.ee> | 2024-04-28 13:59:57 +0200 |
| commit | a9a69a5a312f889d95c60de52485ee8e96e20e47 (patch) | |
| tree | 06b92c6c9f05e7e223c51bcc8219da52ccb93688 | |
| parent | 31327c2d075a413749c1461c06382993b9bba90e (diff) | |
| download | ffmpeg-a9a69a5a312f889d95c60de52485ee8e96e20e47.tar.gz | |
avcodec/aacenc: don't redundantly re-compute max sfb
Remove an unneeded inner loop in adjust_frame_information that
had no effect, the loop body can be run only once and will compute
the same max sfb.
| -rw-r--r-- | libavcodec/aacenc.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/libavcodec/aacenc.c b/libavcodec/aacenc.c index 7feb723289..163598e938 100644 --- a/libavcodec/aacenc.c +++ b/libavcodec/aacenc.c @@ -538,11 +538,9 @@ static void adjust_frame_information(ChannelElement *cpe, int chans) maxsfb = 0; cpe->ch[ch].pulse.num_pulse = 0; for (w = 0; w < ics->num_windows; w += ics->group_len[w]) { - for (w2 = 0; w2 < ics->group_len[w]; w2++) { - for (cmaxsfb = ics->num_swb; cmaxsfb > 0 && cpe->ch[ch].zeroes[w*16+cmaxsfb-1]; cmaxsfb--) - ; - maxsfb = FFMAX(maxsfb, cmaxsfb); - } + for (cmaxsfb = ics->num_swb; cmaxsfb > 0 && cpe->ch[ch].zeroes[w*16+cmaxsfb-1]; cmaxsfb--) + ; + maxsfb = FFMAX(maxsfb, cmaxsfb); } ics->max_sfb = maxsfb; |