diff mbox

[FFmpeg-devel,5/7] lavfi: add outlink helper functions.

Message ID 20170717141926.7442-5-george@nsup.org
State Accepted
Commit ed1c884b9e0d7f63199d5d47350b1fa65374d126
Headers show

Commit Message

Nicolas George July 17, 2017, 2:19 p.m. UTC
These wrappers cost nothing, they make the namespace more
consistent and they will be useful if/when locking becomes
necessary.

Signed-off-by: Nicolas George <george@nsup.org>
---
 libavfilter/filters.h | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

Comments

Michael Niedermayer July 18, 2017, 12:51 a.m. UTC | #1
On Mon, Jul 17, 2017 at 04:19:24PM +0200, Nicolas George wrote:
> These wrappers cost nothing, they make the namespace more
> consistent and they will be useful if/when locking becomes
> necessary.
> 
> Signed-off-by: Nicolas George <george@nsup.org>
> ---
>  libavfilter/filters.h | 21 +++++++++++++++++++++
>  1 file changed, 21 insertions(+)
> 
> diff --git a/libavfilter/filters.h b/libavfilter/filters.h
> index 370d99b38b..1cbc18158f 100644
> --- a/libavfilter/filters.h
> +++ b/libavfilter/filters.h
> @@ -26,6 +26,7 @@
>   */
>  
>  #include "avfilter.h"
> +#include "internal.h"
>  
>  /**
>   * Special return code when activate() did not do anything.
> @@ -139,4 +140,24 @@ int ff_inlink_acknowledge_status(AVFilterLink *link, int *rstatus, int64_t *rpts
>   */
>  void ff_inlink_request_frame(AVFilterLink *link);
>  
> +/**
> + * Test if a frame is wanted on an output link.
> + */
> +static inline int ff_outlink_frame_wanted(AVFilterLink *link)
> +{
> +    return link->frame_wanted_out;
> +}

LGTM

[...]
diff mbox

Patch

diff --git a/libavfilter/filters.h b/libavfilter/filters.h
index 370d99b38b..1cbc18158f 100644
--- a/libavfilter/filters.h
+++ b/libavfilter/filters.h
@@ -26,6 +26,7 @@ 
  */
 
 #include "avfilter.h"
+#include "internal.h"
 
 /**
  * Special return code when activate() did not do anything.
@@ -139,4 +140,24 @@  int ff_inlink_acknowledge_status(AVFilterLink *link, int *rstatus, int64_t *rpts
  */
 void ff_inlink_request_frame(AVFilterLink *link);
 
+/**
+ * Test if a frame is wanted on an output link.
+ */
+static inline int ff_outlink_frame_wanted(AVFilterLink *link)
+{
+    return link->frame_wanted_out;
+}
+
+/**
+ * Set the status field of a link from the source filter.
+ * The pts should reflect the timestamp of the status change,
+ * in link time base and relative to the frames timeline.
+ * In particular, for AVERROR_EOF, it should reflect the
+ * end time of the last frame.
+ */
+static inline void ff_outlink_set_status(AVFilterLink *link, int status, int64_t pts)
+{
+    ff_avfilter_link_set_in_status(link, status, pts);
+}
+
 #endif /* AVFILTER_FILTERS_H */