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

Michael Niedermayer michael at niedermayer.cc
Sat Aug 7 22:30:32 EEST 2021


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 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.
> 
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>
> ---
>  libavcodec/acelp_vectors.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)

LGTM

thx

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Good people do not need laws to tell them to act responsibly, while bad
people will find a way around the laws. -- Plato
-------------- 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/20210807/2a5e3023/attachment.sig>


More information about the ffmpeg-devel mailing list