[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