[FFmpeg-devel] [PATCH 1/3] libavfilter/vf_yadif: Make frame management logic and options shareable

[Michael Niedermayer]

On Fri, Oct 26, 2018 at 08:56:04AM -0700, Philip Langdale wrote:
> I'm writing a cuda implementation of yadif, and while this
> obviously has a very different implementation of the actual
> filtering, all the frame management is unchanged. To avoid
> duplicating that logic, let's make it shareable.
> 
> From the perspective of the existing filter, the only real change
> is introducing a function pointer for the filter() function so it
> can be specified for the specific filter.
> 
> Signed-off-by: Philip Langdale <philipl at overt.org>
> ---
>  libavfilter/Makefile       |   2 +-
>  libavfilter/vf_yadif.c     | 190 +--------------------------------
>  libavfilter/yadif.h        |   9 ++
>  libavfilter/yadif_common.c | 209 +++++++++++++++++++++++++++++++++++++
>  4 files changed, 222 insertions(+), 188 deletions(-)
>  create mode 100644 libavfilter/yadif_common.c
[...]

> +#define OFFSET(x) offsetof(YADIFContext, x)
> +#define FLAGS AV_OPT_FLAG_VIDEO_PARAM|AV_OPT_FLAG_FILTERING_PARAM
> +
> +#define CONST(name, help, val, unit) { name, help, 0, AV_OPT_TYPE_CONST, {.i64=val}, INT_MIN, INT_MAX, FLAGS, unit }
> +

> +const AVOption yadif_options[] = {

missing prefix

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Take away the freedom of one citizen and you will be jailed, take away
the freedom of all citizens and you will be congratulated by your peers
in Parliament.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20181101/c589443e/attachment.sig>