From b92a744a04a17bfdce7d9fba5e90864375f281ac Mon Sep 17 00:00:00 2001
From: Filippos Karapetis
Date: Sun, 3 Jan 2010 21:53:34 +0000
Subject: 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
---
 engines/sci/engine/klists.cpp | 8 --------
 1 file changed, 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;
-- 
cgit v1.2.3