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

Michael Niedermayer michaelni at gmx.at
Sat Aug 13 22:37:01 CEST 2011


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.

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

Concerning the gods, I have no means of knowing whether they exist or not
or of what sort they may be, because of the obscurity of the subject, and
the brevity of human life -- Protagoras
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20110813/f70190c0/attachment.asc>


More information about the ffmpeg-devel mailing list