[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