[FFmpeg-devel] [PATCH 14/17] lavfi: disallow ff_request_frame for filters using activate.

[Nicolas George]

Having two different functions allows to have stricter tests
and detect errors earlier.

Signed-off-by: Nicolas George <george at nsup.org>
---
 libavfilter/avfilter.c | 1 +
 libavfilter/internal.h | 3 +++
 2 files changed, 4 insertions(+)


Unchanged.


diff --git a/libavfilter/avfilter.c b/libavfilter/avfilter.c
index d7677d5f23..680ecf8f95 100644
--- a/libavfilter/avfilter.c
+++ b/libavfilter/avfilter.c
@@ -407,6 +407,7 @@ int ff_request_frame(AVFilterLink *link)
 {
     FF_TPRINTF_START(NULL, request_frame); ff_tlog_link(NULL, link, 1);
 
+    av_assert1(!link->dst->filter->activate);
     if (link->status_out)
         return link->status_out;
     if (link->status_in) {
diff --git a/libavfilter/internal.h b/libavfilter/internal.h
index 8e9aadd40b..69048144f7 100644
--- a/libavfilter/internal.h
+++ b/libavfilter/internal.h
@@ -310,6 +310,9 @@ int ff_poll_frame(AVFilterLink *link);
 /**
  * Request an input frame from the filter at the other end of the link.
  *
+ * This function must not be used by filters using the activate callback,
+ * use ff_link_set_frame_wanted() instead.
+ *
  * The input filter may pass the request on to its inputs, fulfill the
  * request from an internal buffer or any other means specific to its function.
  *
-- 
2.11.0