From 29e696c9c010778b4750efb42c3e89544d56ec4e Mon Sep 17 00:00:00 2001 From: Marton Balint Date: Sun, 22 Jun 2025 12:08:46 +0200 Subject: [PATCH] avfilter/filters: simplify FF_FILTER_FORWARD_WANTED_ANY The status check is unneeded because an outlink with a nonzero status should always return 0 for ff_outlink_frame_wanted(). Also use unsigned for index because nb_outputs is unsigned as well. Signed-off-by: Marton Balint --- libavfilter/filters.h | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/libavfilter/filters.h b/libavfilter/filters.h index 66dc611a98..e3013eda65 100644 --- a/libavfilter/filters.h +++ b/libavfilter/filters.h @@ -701,9 +701,7 @@ static inline void ff_outlink_set_status(AVFilterLink *link, int status, int64_t * If the flag is set on any of the outputs, this macro will return immediately. */ #define FF_FILTER_FORWARD_WANTED_ANY(filter, inlink) do { \ - for (int i = 0; i < filter->nb_outputs; i++) { \ - if (ff_outlink_get_status(filter->outputs[i])) \ - continue; \ + for (unsigned i = 0; i < filter->nb_outputs; i++) { \ if (ff_outlink_frame_wanted(filter->outputs[i])) { \ ff_inlink_request_frame(inlink); \ return 0; \