[MPlayer-dev-eng] [PATCH] Factorize ROUND() redefinitions macro

Diego Biurrun diego at biurrun.de
Mon Jan 3 20:29:28 CET 2011


On Sat, Dec 25, 2010 at 05:41:22PM +0100, Clément Bœsch wrote:
> 
> ROUND() is redefined in a few places in the code, so here is a patch to
> factorize them in mpcommon.h. Is it the correct place to put it? I don't
> see any macro in it so…

I don't know a better place offhand, so mpcommon.h should do for now.

> --- a/mpcommon.h
> +++ b/mpcommon.h
> @@ -25,6 +25,8 @@
>  
> +#define ROUND(x) ((int)((x)<0 ? (x)-0.5 : (x)+0.5))

This would be slightly more readable with spaces around the operators.

Apply anytime.

Diego


More information about the MPlayer-dev-eng mailing list