[FFmpeg-devel] [PATCH] overlay filter

[Stefano Sabatini]

On date Thursday 2010-11-18 16:19:45 -0800, Baptiste Coudurier encoded:
> Hi Stefano,
> 
> On 11/15/2010 04:08 AM, Stefano Sabatini wrote:
> >On date Friday 2010-11-12 15:12:06 +0100, Stefano Sabatini encoded:
> >>On date Friday 2010-11-12 13:36:46 +0100, Stefano Sabatini encoded:
> >>>On date Friday 2010-07-16 13:29:42 -0700, Baptiste Coudurier encoded:
> >>>>On 07/16/2010 10:02 AM, Stefano Sabatini wrote:
> >>>>>Hi all,
> >>>>>this is a work in progress based on Bobby and Baptiste work.
> >>>>>
> >>>>>It's not yet ready for review, I'm sending it here in case someone
> >>>>>wants to have a look at it and/or commenting it.
> >>>>>
> >>>>
> >>>>This filter is _highly_ inefficient in this form. Copying the source
> >>>>is useless and can be avoided. Furthermore the timestamps handling
> >>>>is wrong.
> >>>>
> >>>>What you have to do is to handle timestamps correctly in my version.
> >>>
> >>>Done, also this version works with movie-in-movie (but supporting loop
> >>>in movie seems a simple and useful feature anyway).
> >>>
> >>>There is still a bug related to direct rendering (and so only visible
> >>>with ffplay), looking into that
> >>[...]
> >>
> >>Fixed by adding AV_PERM_PRESERVE flag in the rej_perms.
> >
> >Updated with some docs fixes.
> 
> Thanks for the work.
> 
> Does the filter works to overlay a single picture logo in this form ?
> I believe that's a needed feature and I'll work on this.

Of course it is supported:
"movie=0:png:ffmpeg.png, scale=80:-1 [over]; [in][over] overlay [out]"

that's for showing a nice overlayed logo for the next FFmpeg show at
linuxtag/whatever ;-)
 
Note that I'm testing against the movie source I posted here, AFAIK
timestamp handling in the soc repo movie source is broken.

> Somebody else please review the code, I'd like some comments on the
> average alpha section for subsampled yuv.
-- 
FFmpeg = Fantastic and Free Meaningful Perennial EnGine