Re: [FFmpeg-devel] [PATCH 4/4] avcodec/acelp_vectors: Add missing brackets

2021-08-07 Thread Michael Niedermayer
On Fri, Aug 06, 2021 at 06:17:25PM +0200, Andreas Rheinhardt wrote: > 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 comm

[FFmpeg-devel] [PATCH 4/4] avcodec/acelp_vectors: Add missing brackets

2021-08-06 Thread Andreas Rheinhardt
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 cond