diff mbox

[FFmpeg-devel,4/7] lavfi: make FFERROR_NOT_READY available to filters.

Message ID 20170717141926.7442-4-george@nsup.org
State Accepted
Commit b77f041dff5f669d1d297697a34023de1c33a33f
Headers show

Commit Message

Nicolas George July 17, 2017, 2:19 p.m. UTC
I am not entirely sure that this return code is useful,
but having and using it makes no harm.

Signed-off-by: Nicolas George <george@nsup.org>
---
 libavfilter/avfilter.c | 2 --
 libavfilter/filters.h  | 5 +++++
 2 files changed, 5 insertions(+), 2 deletions(-)

Comments

Michael Niedermayer July 18, 2017, 12:43 a.m. UTC | #1
On Mon, Jul 17, 2017 at 04:19:23PM +0200, Nicolas George wrote:
> I am not entirely sure that this return code is useful,
> but having and using it makes no harm.
> 
> Signed-off-by: Nicolas George <george@nsup.org>
> ---
>  libavfilter/avfilter.c | 2 --
>  libavfilter/filters.h  | 5 +++++
>  2 files changed, 5 insertions(+), 2 deletions(-)

LGTM

thx

[...]
diff mbox

Patch

diff --git a/libavfilter/avfilter.c b/libavfilter/avfilter.c
index e60b0247bc..185ba8df00 100644
--- a/libavfilter/avfilter.c
+++ b/libavfilter/avfilter.c
@@ -1304,8 +1304,6 @@  static int forward_status_change(AVFilterContext *filter, AVFilterLink *in)
     return 0;
 }
 
-#define FFERROR_NOT_READY FFERRTAG('N','R','D','Y')
-
 static int ff_filter_activate_default(AVFilterContext *filter)
 {
     unsigned i;
diff --git a/libavfilter/filters.h b/libavfilter/filters.h
index 2c78d60e62..370d99b38b 100644
--- a/libavfilter/filters.h
+++ b/libavfilter/filters.h
@@ -28,6 +28,11 @@ 
 #include "avfilter.h"
 
 /**
+ * Special return code when activate() did not do anything.
+ */
+#define FFERROR_NOT_READY FFERRTAG('N','R','D','Y')
+
+/**
  * Mark a filter ready and schedule it for activation.
  *
  * This is automatically done when something happens to the filter (queued