Re: [PATCH] Define/use ALIGN() instead of open coding it

2010-04-09 Thread Mark Kettenis
Please use a different name for the macro. On *BSD ALIGN() is a macro that gets pulled in as part of sys/param.h and will conflict with your new macro. ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info:

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

2010-04-08 Thread Matt Turner
Signed-off-by: Matt Turner matts...@gmail.com --- Xext/xf86bigfont.c|2 +- hw/dmx/dmxpict.c |2 +- hw/kdrive/ephyr/XF86dri.c |6 +++--- hw/xfree86/dixmods/extmod/xf86vmode.c |6 +++--- hw/xfree86/int10/generic.c|

Re: [PATCH] Define/use ALIGN() instead of open coding it

2010-04-08 Thread Keith Packard
On Thu, 8 Apr 2010 19:10:52 -0400, Matt Turner matts...@gmail.com wrote: -#define GET_HIGH_BASE(x) (((V_BIOS + (x) + getpagesize() - 1)/getpagesize()) \ - * getpagesize()) +#define GET_HIGH_BASE(x) (ALIGN(V_BIOS + (x), getpagesize())) #endif Note that this