[FFmpeg-devel] [PATCH 2/2] avutil/film_grain_params: remove do loop in CHECK macro
Leo Izen
leo.izen at gmail.com
Sun Mar 24 02:10:14 EET 2024
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>
---
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);
--
2.44.0
More information about the ffmpeg-devel
mailing list