[FFmpeg-cvslog] avutil/film_grain_params: remove do loop in CHECK macro

Leo Izen git at videolan.org
Sun Mar 24 16:53:49 EET 2024


ffmpeg | branch: master | Leo Izen <leo.izen at gmail.com> | Sat Mar 23 20:10:14 2024 -0400| [ac21582e5334388bc0ac47c273ac134023c64984] | committer: Leo Izen

avutil/film_grain_params: remove do loop in CHECK macro

The continue statement will break out of the do/while loop, not the
outer loop as intended. This is one (compound) statement anyway, so we
can remove the do/while entirely.

Signed-off-by: Leo Izen <leo.izen at gmail.com>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=ac21582e5334388bc0ac47c273ac134023c64984
---

 libavutil/film_grain_params.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/libavutil/film_grain_params.c b/libavutil/film_grain_params.c
index b3fa37f527..8f8dcee569 100644
--- a/libavutil/film_grain_params.c
+++ b/libavutil/film_grain_params.c
@@ -70,10 +70,8 @@ const AVFilmGrainParams *av_film_grain_params_select(const AVFrame *frame)
             continue;
 
 #define CHECK(a, b, unspec)                                     \
-    do {                                                        \
         if ((a) != (unspec) && (b) != (unspec) && (a) != (b))   \
-            continue;                                           \
-    } while (0)
+            continue
 
         CHECK(fgp->bit_depth_luma,   bit_depth_luma,         0);
         CHECK(fgp->bit_depth_chroma, bit_depth_chroma,       0);



More information about the ffmpeg-cvslog mailing list