[FFmpeg-devel] [PATCH] lavfi: port libmpcodecs delogo filter

Stefano Sabatini stefano.sabatini-lala at poste.it
Sun Aug 14 12:30:40 CEST 2011


On date Saturday 2011-08-13 22:37:01 +0200, Michael Niedermayer encoded:
> On Sat, Aug 13, 2011 at 06:15:58PM +0200, Stefano Sabatini wrote:
> > On date Thursday 2011-08-11 15:33:35 +0200, Stefano Sabatini encoded:
> > > On date Thursday 2011-08-11 14:23:55 +0200, Michael Niedermayer encoded:
> > > > On Thu, Aug 11, 2011 at 01:47:20PM +0200, Stefano Sabatini wrote:
> > > > [...]
> > > > > I'd apply the attached version with named options parsing, but if
> > > > > someone insists I'll try to implement this idea to retain backward
> > > > > compatibility with the MPlayer delogo filter syntax.
> > > > 
> > > > i think we should aim at maintaining compatibility with the syntax
> > > > used by libmpcodecs filters
> > > 
> > > Updated, with old and new syntax (main drawback is a not very clear
> > > error message in case the user specifies a wrong number of arguments
> > > with the x:y:w:h:band syntax). If considered overkill I'll commit the
> > > previous version.
> > [...]
> > 
> > Updated with a few cosmetic fixes. I'm going to apply in a few days
> > (and remove the mp=delogo filter) if I see no comments.
> 
> if it gives same output and runs at same speed theres not so much that
> can be wrong.

Pushed, and mp wrapper removed.
-- 
FFmpeg = Fast and Free Merciful Puritan Ecstatic Goblin


More information about the ffmpeg-devel mailing list