[FFmpeg-devel] [PATCH] lavfi: add histeq filter

[Nicolas George]

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.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130104/e575f0e9/attachment.asc>