[FFmpeg-devel] [PATCH] [SOCIS] [WIP] libavfilter: add vf_hue.c

[Nicolas George]

Le decadi 20 thermidor, an CCXX, Stefano Sabatini a écrit :
> So the question is: should we rely on this behavior?
> 
> The plain C variant would be:
>             if (new_x & 0x300)
>                 new_x = av_clip_uint8(new_x);

Since we have better, there is clearly no need to keep the hack.

The test could become "new_x & ~0xFF". Or even go away completely, as
av_clip_uint8 does exactly that.

(As a side note, av_clip_uint8_c() is implemented exactly that way.)

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/20120806/2033e5fd/attachment.asc>