From 127da193d4935b44dc70e0fe0c342f7d35908180 Mon Sep 17 00:00:00 2001 From: Andreas Rheinhardt Date: Fri, 6 Aug 2021 18:10:19 +0200 Subject: [PATCH] avcodec/acelp_vectors: Add missing brackets Before 3793caa5e2d1d16ed45771574b2ffc932497cfcf the code was "if (...) do { ... } while (...);". After said commit this became "if (...) av_assert0(...); do { ... } while (...);", i.e. the loop is always executed. This commit changes the logic to what it was before said commit. Notice that the condition is always true in FATE, so no changes are necessary there. This fixes a -Wmisleading-indentation warning from GCC 11. Reviewed-by: Michael Niedermayer Signed-off-by: Andreas Rheinhardt --- libavcodec/acelp_vectors.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavcodec/acelp_vectors.c b/libavcodec/acelp_vectors.c index 0d4de0e4c3..8adea683c1 100644 --- a/libavcodec/acelp_vectors.c +++ b/libavcodec/acelp_vectors.c @@ -229,13 +229,14 @@ void ff_set_fixed_vector(float *out, const AMRFixed *in, float scale, int size) int x = in->x[i], repeats = !((in->no_repeat_mask >> i) & 1); float y = in->y[i] * scale; - if (in->pitch_lag > 0) + if (in->pitch_lag > 0) { av_assert0(x < size); do { out[x] += y; y *= in->pitch_fac; x += in->pitch_lag; } while (x < size && repeats); + } } }