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/spl-util.cc | |
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/spl-util.cc')
-rw-r--r-- | crawl-ref/source/spl-util.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/crawl-ref/source/spl-util.cc b/crawl-ref/source/spl-util.cc index b727117eb8..c3097b0e53 100644 --- a/crawl-ref/source/spl-util.cc +++ b/crawl-ref/source/spl-util.cc @@ -92,7 +92,7 @@ void init_spell_descs(void) { spell_desc &data = spelldata[i]; -#if DEBUG +#ifdef DEBUG if (data.id < SPELL_NO_SPELL || data.id >= NUM_SPELLS) end(1, false, "spell #%d has invalid id %d", i, data.id); |