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

Mark Himsley mark at mdsh.com
Sun Nov 6 17:56:09 CET 2011


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

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: ffmpeg-fade-alpha-003b.diff
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20111106/47a01c4a/attachment.ksh>


More information about the ffmpeg-devel mailing list