Having two different functions allows to have stricter tests and detect errors earlier.
Signed-off-by: Nicolas George <geo...@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 _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel