From c6695b1d6aa778575f4cefc1942253cbb8a65fbf Mon Sep 17 00:00:00 2001
From: Martin Kiewitz <m_kiewitz@users.sourceforge.net>
Date: Wed, 30 Jun 2010 10:31:23 +0000
Subject: [PATCH] SCI: removing all the remaining debug output from signature
 matching

svn-id: r50525
---
 engines/sci/engine/kernel.cpp | 25 +++++++++++--------------
 1 file changed, 11 insertions(+), 14 deletions(-)

diff --git a/engines/sci/engine/kernel.cpp b/engines/sci/engine/kernel.cpp
index c7a7b4e843a..2eeefe83d93 100644
--- a/engines/sci/engine/kernel.cpp
+++ b/engines/sci/engine/kernel.cpp
@@ -771,7 +771,11 @@ void Kernel::signatureDebug(const char *sig, int argc, const reg_t *argv) {
 		if (argc) {
 			reg_t parameter = *argv;
 			printf("%04x:%04x (", PRINT_REG(parameter));
-			kernelSignatureDebugType(findRegType(parameter));
+			int regType = findRegType(parameter);
+			if (regType)
+				kernelSignatureDebugType(regType);
+			else
+				printf("unknown type of %04x:%04x", PRINT_REG(parameter));
 			printf(")");
 			argv++;
 			argc--;
@@ -803,15 +807,11 @@ bool Kernel::signatureMatch(const char *sig, int argc, const reg_t *argv) {
 		if ((*sig & KSIG_ANY) != KSIG_ANY) {
 			int type = findRegType(*argv);
 
-			if (!type) {
-				warning("[KERNEL] call signature: couldn't determine type of ref %04x:%04x", PRINT_REG(*argv));
-				return false;
-			}
+			if (!type)
+				return false; // couldn't determine type
 
-			if (!(type & *sig)) {
-				warning("[KERNEL] call signature: %d args left, is %d, should be %d", argc, type, *sig);
-				return false;
-			}
+			if (!(type & *sig))
+				return false; // type mismatch
 
 		}
 		if (!(*sig & KSIG_ELLIPSIS))
@@ -820,15 +820,12 @@ bool Kernel::signatureMatch(const char *sig, int argc, const reg_t *argv) {
 		--argc;
 	}
 
-	if (argc) {
-		warning("[KERNEL] call signature: too many arguments");
+	if (argc)
 		return false; // Too many arguments
-	}
 	if (*sig == 0 || (*sig & KSIG_ELLIPSIS))
 		return true;
 
-	warning("[KERNEL] call signature: too few arguments");
-	return false;
+	return false; // Too few arguments
 }
 
 void Kernel::setDefaultKernelNames() {