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 <[email protected]> --- 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 [email protected] https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email [email protected] with subject "unsubscribe".
