[PATCH] Define/use X_ALIGN2() instead of open coding it

Michel Dänzer michel at daenzer.net
Mon Jun 21 01:33:40 PDT 2010


On Sam, 2010-06-19 at 18:14 -0400, Matt Turner wrote: 
> diff --git a/exa/exa_priv.h b/exa/exa_priv.h
> index 58d04b0..0d2c179 100644
> --- a/exa/exa_priv.h
> +++ b/exa/exa_priv.h
> @@ -273,7 +273,7 @@ extern DevPrivateKeyRec exaGCPrivateKeyRec;
>  #define EXA_ALIGN(offset, align) (((offset) + (align) - 1) - \
>  	(((offset) + (align) - 1) % (align)))
>  /** Align an offset to a power-of-two alignment */
> -#define EXA_ALIGN2(offset, align) (((offset) + (align) - 1) & ~((align) - 1))
> +#define EXA_ALIGN2(offset, align) X_ALIGN2(offset, align)

I'd have suggested dropping EXA_ALIGN2() in favour of the new thing, but
I can't find any uses of it anyway, so I'd just drop it.


-- 
Earthling Michel Dänzer           |                http://www.vmware.com
Libre software enthusiast         |          Debian, X and DRI developer


More information about the xorg-devel mailing list