[FFmpeg-devel] [PATCH] yadif port to libavfitler

Stefano Sabatini stefano.sabatini-lala
Sat Sep 25 17:09:23 CEST 2010


On date Saturday 2010-09-25 15:56:02 +0200, Michael Niedermayer encoded:
> On Sat, Sep 25, 2010 at 11:11:12AM +0200, Stefano Sabatini wrote:
> > On date Wednesday 2010-09-08 17:50:59 +0200, Stefano Sabatini encoded:
> > > On date Saturday 2010-09-04 12:45:08 +0200, Stefano Sabatini encoded:
> > > [...]
> > > > Updated against the av_get_cpu_flags() patch.
> > > > 
> > > > Still missing documentation (Baptiste feel free to continue to work on
> > > > this, I don't even know how did you test it).
> > > 
> > > Updated again. Documentation for the second param should be extended,
> > > (Mplayer docs is outdated as well), also maybe ASM code should be
> > > placed in /ARCH dirs.
> > > 
> > > I'll let someone else work on this though, so don't expect other
> > > updated patches from me.
> > 
> > Updated, I moved the x86 code to a new libavfilter/x86 dir, and added
> 
> now it doesnt compile anymore
> libavfilter/x86/yadif.c: In function ?ff_yadif_filter_line_mmx?:
> libavfilter/x86/yadif.c:86: error: ?YADIFContext? undeclared (first use in this function)
> libavfilter/x86/yadif.c:86: error: (Each undeclared identifier is reported only once
> libavfilter/x86/yadif.c:86: error: for each function it appears in.)
> libavfilter/x86/yadif.c:86: error: ?yadif? undeclared (first use in this function)

Fixed.

BTW, can someone suggest some way to test it? What about adding an
interlacer, libmpcodecs/tinterlace would be a good option?

Regards.
-- 
FFmpeg = Formidable and Freak MultiPurpose Elfic Glue



More information about the ffmpeg-devel mailing list