[FFmpeg-devel] [PATCH] lavfi: add histeq filter
Stefano Sabatini
stefasab at gmail.com
Fri Jan 4 16:42:10 CET 2013
On date Friday 2013-01-04 16:19:07 +0100, Nicolas George encoded:
> Le quintidi 15 nivôse, an CCXXI, Clement Boesch a écrit :
> > > + if (((m = FFMAX(r, FFMAX(g, b))) * lut) / luma > 255) {
> > nit: I'm not sure the double expansion of the FFMAX macro will generate a
> > sexy asm.
>
> We have a FFMAX3 (but it does mostly the same thing).
>
> > > + .rej_perms = AV_PERM_PRESERVE
> > Is that necessary?
>
> Since the permissions cleanup last summer, .rej_perms on inputs is usually
> not useful.
Updated.
Will push tomorrow if I see no more comments.
--
FFmpeg = Foolish and Freak Minimalistic Pitiful Evanescent Guide
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-lavfi-add-histeq-filter.patch
Type: text/x-diff
Size: 17157 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130104/38acd219/attachment.bin>
More information about the ffmpeg-devel
mailing list