diff options
| author | Steven Noonan <steven@uplinklabs.net> | 2009-10-13 03:05:10 -0700 |
|---|---|---|
| committer | Steven Noonan <steven@uplinklabs.net> | 2009-10-13 06:23:13 -0700 |
| commit | 997a4469a2022461f5fe011bd7d0843d6dd6c5de (patch) | |
| tree | 96e1c6a217e1c15852ab651361e8085d0a27e911 /crawl-ref/source/debug.h | |
| parent | 3f6c3a3969efce77a980fdf7e3c76183bcad2db3 (diff) | |
| download | crawl-ref-997a4469a2022461f5fe011bd7d0843d6dd6c5de.tar.gz crawl-ref-997a4469a2022461f5fe011bd7d0843d6dd6c5de.zip | |
fix 'DEBUG' macro usage consistency
Sometimes we were doing #if DEBUG and others we were doing #ifdef DEBUG.
If we mix both, we have problems: If the DEBUG macro isn't defined, the
statement '#if DEBUG' doesn't really make sense logically, because 'DEBUG'
has no value. And if we '#define DEBUG 0', then the '#ifdef DEBUG's
become true statements.
The easiest fix is to swap out the #ifs with #ifdefs.
Signed-off-by: Steven Noonan <steven@uplinklabs.net>
Diffstat (limited to 'crawl-ref/source/debug.h')
| -rw-r--r-- | crawl-ref/source/debug.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/crawl-ref/source/debug.h b/crawl-ref/source/debug.h index e75903c81f..44571a1891 100644 --- a/crawl-ref/source/debug.h +++ b/crawl-ref/source/debug.h @@ -11,11 +11,11 @@ #include "enum.h" // Synch with ANSI definitions. -#if DEBUG && defined(NDEBUG) +#if defined(DEBUG) && defined(NDEBUG) #error DEBUG and NDEBUG are out of sync! #endif -#if !DEBUG && !defined(NDEBUG) +#if !defined(DEBUG) && !defined(NDEBUG) #error DEBUG and NDEBUG are out of sync! #endif @@ -36,7 +36,7 @@ #define COMPILE_CHECK(expr, tag) #endif -#if DEBUG +#ifdef DEBUG void AssertFailed(const char *expr, const char *file, int line); |