sane_nodep() and sane_listp() should not be dummy functions when DISABLE_VALIDATIONS is set, so that the engine doesn't try and refer to null pointer objects (and subsequently crash)

svn-id: r46956

1 files changed, 0 insertions, 8 deletions

diff --git a/engines/sci/engine/klists.cpp b/engines/sci/engine/klists.cpp
index 2b2a9b4562..5a24ee7698 100644
--- a/engines/sci/engine/klists.cpp
+++ b/engines/sci/engine/klists.cpp
@@ -28,13 +28,6 @@
 
 namespace Sci {
 
-#ifdef DISABLE_VALIDATIONS
-
-#define sane_nodep(a, b) 1
-#define sane_listp(a, b) 1
-
-#else
-
 static int sane_nodep(EngineState *s, reg_t addr) {
 	int have_prev = 0;
 	reg_t prev = addr;
@@ -89,7 +82,6 @@ int sane_listp(EngineState *s, reg_t addr) {
 
 	return 1; // Empty list is fine
 }
-#endif
 
 reg_t kNewList(EngineState *s, int argc, reg_t *argv) {
 	reg_t listbase;