[FFmpeg-devel] [PATCH 02/12] avfilter/filters: simplify FF_FILTER_FORWARD_WANTED_ANY

From: Marton Balint <cus@passwd.hu>
To: ffmpeg-devel@ffmpeg.org
Cc: Marton Balint <cus@passwd.hu>
Subject: [FFmpeg-devel] [PATCH 02/12] avfilter/filters: simplify FF_FILTER_FORWARD_WANTED_ANY
Date: Tue, 24 Jun 2025 21:23:00 +0200
Message-ID: <20250624192318.7430-2-cus@passwd.hu> (raw)
In-Reply-To: <20250624192318.7430-1-cus@passwd.hu>

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 <cus@passwd.hu>
---
 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; \
-- 
2.43.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-request@ffmpeg.org with subject "unsubscribe".
