[FFmpeg-devel] [PATCH] vf_fade: fade alpha

Stefano Sabatini stefasab at gmail.com
Thu Nov 10 01:48:39 CET 2011


On date Monday 2011-11-07 18:54:52 +0000, Mark Himsley encoded:
> On 07/11/11 17:01, Stefano Sabatini wrote:
> >On date Sunday 2011-11-06 16:56:09 +0000, Mark Himsley encoded:
> >>On 06/11/2011 16:54, Mark Himsley wrote:
> >>>Add alpha fading.
> >>>
> >>>On 06/11/2011 00:40, Stefano Sabatini wrote:
> >>>>On date Saturday 2011-11-05 21:53:35 +0000, Mark Himsley encoded:
> >>>>>On 05/11/2011 14:26, Stefano Sabatini wrote:
> >>>>>>On date Saturday 2011-11-05 02:01:02 +0000, Mark Himsley encoded:
> >>>>>>>add alpha fading
> >>>>[...]
> >>>>>
> >>>>>Thanks for the review.
> >>>>
> >>>>Thanks for the patch ;-).
> >>>>
> >>>>>
> >>>>>I'll send updated patches.
> >>>
> >>>Including all suggested changes, including removing spurious double
> >>>alpha_expr variable, simplifying black_level determination, changing new
> >>>function name, meaningful names for variables, reformatting function
> >>>call and adding comments in function call.
> >>>
> >>>I also added macros for R G B A Y U V, although currently only 'A' is
> >>>ever used...
> >>
> >>...attached...
> >>
> >>--
> >>Mark
> >>
> 
> [...]
> 
> >>+    { "alpha",    "fade alpha if it's available on the input",  OFFSET(alpha),      AV_OPT_TYPE_INT,    {.dbl = 0    },        0,        1 },       {NULL},
> 
> [...]
> 
> >>+    fade->alpha = !!fade->alpha;
> >
> >this is not needed, as fade->alpha can only be 1 or 0.
> 
> Oh yes. I forgot about that checking. Thanks.
> 
> [...]
> 
> >>+        fade_plane(y, h, inlink->w,
> >>+                fade->factor, fade->black_level, fade->black_level_scaled,
> >>+                0, 1, // offset&  pixstep for Y plane or RGB packed format
> >>+                fade->bpp, outpic->data[0], outpic->linesize[0]);
> >
> >weird indent (no need to post a new patch for this)
> 
> I'll assume you'd rather 'fade->factor' was directly under 'y'.
> Sorry, I do that next time.
> 
> >Looks fine to me, I'll post in a day or so if I read no more comments.
> 
> Thank you.

"Posted" to the main repo.
-- 
FFmpeg = Faithful and Faithful Mind-dumbing Puritan Enhanced God


More information about the ffmpeg-devel mailing list