[FFmpeg-devel] [PATCH 0/2] Update vf_bwdif to use yadif_common v2

Philip Langdale philipl at overt.org
Thu Nov 15 05:44:51 EET 2018


On Tue, 13 Nov 2018 02:11:15 +0100
Thomas Mundt <tmundt75 at gmail.com> wrote:

> Am So., 11. Nov. 2018 um 20:47 Uhr schrieb Philip Langdale <
> philipl at overt.org>:  
> 
> > vf_bwdif's frame management logic is almost identical to that of
> > yadif. The only difference is that it tracks the first and last
> > fields in a sequence, and that requires slight changes to the
> > common code.
> >
> > Assuming it's reasonable to do that tracking even though yadif
> > doesn't need it, we can then remove all the duplicated logic.
> >
> > v2: Rename enum values as recommened by Thomas Mundt.
> >
> > Philip Langdale (2):
> >   avfilter/yadif_common: Add field type tracking to help bwdif
> >   avfilter/vf_bwdif: Use common yadif frame management logic
> >
> >  libavfilter/bwdif.h             |  34 +----
> >  libavfilter/vf_bwdif.c          | 235
> > +++++--------------------------- libavfilter/x86/vf_bwdif_init.c
> > |   3 +- libavfilter/yadif.h             |  14 ++
> >  libavfilter/yadif_common.c      |  12 +-
> >  5 files changed, 64 insertions(+), 234 deletions(-)
> >  
> 
> Patch lgtm.

Thanks. Pushed.

--phil


More information about the ffmpeg-devel mailing list