From ac21582e5334388bc0ac47c273ac134023c64984 Mon Sep 17 00:00:00 2001 From: Leo Izen Date: Sat, 23 Mar 2024 20:10:14 -0400 Subject: [PATCH] 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 --- 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);