Hi, 2011/5/13 Måns Rullgård <[email protected]>: > "Ronald S. Bultje" <[email protected]> writes: > >> --- >> libswscale/swscale.c | 36 +++++++++++++++++++----------------- >> 1 files changed, 19 insertions(+), 17 deletions(-) >> >> diff --git a/libswscale/swscale.c b/libswscale/swscale.c >> index b63a386..8131927 100644 >> --- a/libswscale/swscale.c >> +++ b/libswscale/swscale.c >> @@ -209,6 +209,7 @@ DECLARE_ALIGNED(8, const uint8_t, dither_8x8_220)[8][8]={ >> }; >> #endif >> >> +#define POW2CLIP(x,max) (((x) & ~max) ? (-(x))>>31 & max : (x)) > > There is already at least one macro similar to this somewhere. > Consolidating them in libavutil would probably be a good idea. > Don't let this hold up the patch, on which I don't have an opinion, > though.
Yeah I copied it verbatim from vp8.c. I can give it a sweet av_ prefix and dump it in libavutil/common.h in a subsequent patch, or maybe even in this one. Ronald _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
