[FFmpeg-devel] [PATCH] avfilter/fade: add color option.

Clément Bœsch u at pkh.me
Sun Nov 10 19:09:04 CET 2013


On Sun, Nov 10, 2013 at 12:29:22PM +0100, Stefano Sabatini wrote:
[...]
> > +#define INTERP(cname, cidx) av_clip_uint8(((c[cidx]<<16) + ((int)p[cname] - (int)c[cidx]) * s->factor + (1<<15)) >> 16)
> 
> nittt: c_idx (or ridx, gidx, ... for consistency)
> 

→ INTERP(c_name, c_idx)

[...]
> LGTM, thanks.

Applied, thanks for the review

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20131110/249de170/attachment.asc>


More information about the ffmpeg-devel mailing list