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.i...@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 _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".