diff options
| author | Eric Anholt <eric@anholt.net> | 2008-04-10 10:33:11 -0700 |
|---|---|---|
| committer | Eric Anholt <eric@anholt.net> | 2008-04-10 10:35:14 -0700 |
| commit | a7e3ad1c6b455bda7c4abb352a20845d1d4574a0 (patch) | |
| tree | 55462578bead4c0f75d1d0002317803241b81891 /hw/xquartz/quartzStartup.c | |
| parent | 7909ebe7f163716520f843fae11ac7bdeffcb57c (diff) | |
Remove NDEBUG (assert() disable) define from the X Server.
A few pieces of code were abusing this define for other purposes, which are
converted to #ifndef DEBUG instead. There should be no ABI consequences
to this change.
The rationale is that having the define in xorg-server.h also disables
assert() drivers, which is unexpected, and also difficult to avoid since
xorg-server.h is included in their config.h, and you can't put a #undef in
config.h. As for removing it from the server instead of moving it to an
internal header, we probably shouldn't have unnecessary assert()s in
critical server paths anyway, and if we do we could #define NDEBUG in the
specific cases needed.
Diffstat (limited to 'hw/xquartz/quartzStartup.c')
| -rw-r--r-- | hw/xquartz/quartzStartup.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/hw/xquartz/quartzStartup.c b/hw/xquartz/quartzStartup.c index e25e15583..5ac3017e1 100644 --- a/hw/xquartz/quartzStartup.c +++ b/hw/xquartz/quartzStartup.c @@ -41,13 +41,7 @@ #include "opaque.h" #include "micmap.h" -#ifdef NDEBUG -#undef NDEBUG #include <assert.h> -#define NDEBUG 1 -#else -#include <assert.h> -#endif char **envpGlobal; // argcGlobal and argvGlobal // are from dix/globals.c |