[FFmpeg-devel] [PATCH 2/4] lavfi/opencl: Handle overlay input formats correctly.

Li, Zhong zhong.li at intel.com
Fri Nov 9 11:18:27 EET 2018


> > > > > > > > The main input may have alpha channel, we just ignore it.
> > > > > > > > Also add some checks for incompatible input formats.
> > > > > > > >
> > > > > > > > Signed-off-by: Ruiling Song <ruiling.song at intel.com>
> > > > > > LGTM.
> > > > > > BTW, could the main input with alpha case be supported?
> > > > >
> > > > > I am not sure what kind of support do you mean?
> > > > > I simply ignore the alpha channel of the main input, and do the
> > > > > alpha blending using the overlay alpha.
> > > > > Before this patch, the filter will do it wrong if the main input
> > > > > has alpha channel. Now it works with this patch.
> > > > >
> > > > > Thanks!
> > > > > Ruiling
> > > >
> > > > I mean support alpha blending with alpha channel of main input
> > > > when no overlay alpha.
> > > I think I got your idea, this patch aims to fix the issues reported by Gyan.
> > > If people really want it (blending using alpha of main input) be
> > > supported, we can add it later.
> > > I am not sure whether this sounds ok?
> > >
> > > Thanks!
> > > Ruiling
> >
> > Sure, sound good.
> 
> Can we merge this patch? Any objection or concern?
> 
> Ruiling

Could you please provide the use case (with an alpha channel main input) to verify it?

If no body against, I prefer to merge it after verification during next a few days. 


More information about the ffmpeg-devel mailing list