[Scummvm-cvs-logs] SF.net SVN: scummvm: [21819] scummvm/trunk/engines/cine

[sev at users.sourceforge.net]

Revision: 21819
Author:   sev
Date:     2006-04-11 20:12:56 -0700 (Tue, 11 Apr 2006)
ViewCVS:  http://svn.sourceforge.net/scummvm/?rev=21819&view=rev

Log Message:
-----------
cleanup

Modified Paths:
--------------
    scummvm/trunk/engines/cine/cine.h
    scummvm/trunk/engines/cine/object.cpp
    scummvm/trunk/engines/cine/script.cpp
Modified: scummvm/trunk/engines/cine/cine.h
===================================================================
--- scummvm/trunk/engines/cine/cine.h	2006-04-12 03:01:06 UTC (rev 21818)
+++ scummvm/trunk/engines/cine/cine.h	2006-04-12 03:12:56 UTC (rev 21819)
@@ -87,6 +87,11 @@
 	kCineDebugScript = 1 << 0
 };
 
+enum {
+	kCmpEQ = (1 << 0),
+	kCmpGT = (1 << 1),
+	kCmpLT = (1 << 2)
+};
 
 
 extern Common::SaveFileManager *g_saveFileMan; // TEMP

Modified: scummvm/trunk/engines/cine/object.cpp
===================================================================
--- scummvm/trunk/engines/cine/object.cpp	2006-04-12 03:01:06 UTC (rev 21818)
+++ scummvm/trunk/engines/cine/object.cpp	2006-04-12 03:12:56 UTC (rev 21819)
@@ -306,82 +306,82 @@
 	}
 }
 
-byte compareObjectParam(byte objIdx, byte param1, int16 param2) {
+byte compareObjectParam(byte objIdx, byte type, int16 value) {
 	byte compareResult = 0;
 
-	switch (param1 - 1) {
+	switch (type - 1) {
 	case 0:
-		if (objectTable[objIdx].x == param2) {
-			compareResult |= 1;
+		if (objectTable[objIdx].x == value) {
+			compareResult |= kCmpEQ;
 		}
 
-		if (objectTable[objIdx].x > param2) {
-			compareResult |= 2;
+		if (objectTable[objIdx].x > value) {
+			compareResult |= kCmpGT;
 		}
 
-		if (objectTable[objIdx].x < param2) {
-			compareResult |= 4;
+		if (objectTable[objIdx].x < value) {
+			compareResult |= kCmpLT;
 		}
 
 		break;
 	case 1:
-		if (objectTable[objIdx].y == param2) {
-			compareResult |= 1;
+		if (objectTable[objIdx].y == value) {
+			compareResult |= kCmpEQ;
 		}
 
-		if (objectTable[objIdx].y > param2) {
-			compareResult |= 2;
+		if (objectTable[objIdx].y > value) {
+			compareResult |= kCmpGT;
 		}
 
-		if (objectTable[objIdx].y < param2) {
-			compareResult |= 4;
+		if (objectTable[objIdx].y < value) {
+			compareResult |= kCmpLT;
 		}
 
 		break;
 	case 2:
-		if (objectTable[objIdx].mask == param2) {
-			compareResult |= 1;
+		if (objectTable[objIdx].mask == value) {
+			compareResult |= kCmpEQ;
 		}
 
-		if (objectTable[objIdx].mask > param2) {
-			compareResult |= 2;
+		if (objectTable[objIdx].mask > value) {
+			compareResult |= kCmpGT;
 		}
 
-		if (objectTable[objIdx].mask < param2) {
-			compareResult |= 4;
+		if (objectTable[objIdx].mask < value) {
+			compareResult |= kCmpLT;
 		}
 
 		break;
 	case 3:
-		if (objectTable[objIdx].frame == param2) {
-			compareResult |= 1;
+		if (objectTable[objIdx].frame == value) {
+			compareResult |= kCmpEQ;
 		}
 
-		if (objectTable[objIdx].frame > param2) {
-			compareResult |= 2;
+		if (objectTable[objIdx].frame > value) {
+			compareResult |= kCmpGT;
 		}
 
-		if (objectTable[objIdx].frame < param2) {
-			compareResult |= 4;
+		if (objectTable[objIdx].frame < value) {
+			compareResult |= kCmpLT;
 		}
 
 		break;
 	case 4:
-		if (objectTable[objIdx].costume == param2) {
-			compareResult |= 1;
+		if (objectTable[objIdx].costume == value) {
+			compareResult |= kCmpEQ;
 		}
 
-		if (objectTable[objIdx].costume > param2) {
-			compareResult |= 2;
+		if (objectTable[objIdx].costume > value) {
+			compareResult |= kCmpGT;
 		}
 
-		if (objectTable[objIdx].costume < param2) {
-			compareResult |= 4;
+		if (objectTable[objIdx].costume < value) {
+			compareResult |= kCmpLT;
 		}
 
 		break;
 	default:
-		error("Unsupported compare type: %d in compareObjectParam", param1 - 1);
+		error("Unsupported compare type: %d in compareObjectParam", type - 1);
 	}
 
 	return compareResult;

Modified: scummvm/trunk/engines/cine/script.cpp
===================================================================
--- scummvm/trunk/engines/cine/script.cpp	2006-04-12 03:01:06 UTC (rev 21818)
+++ scummvm/trunk/engines/cine/script.cpp	2006-04-12 03:12:56 UTC (rev 21819)
@@ -34,12 +34,6 @@
 
 namespace Cine {
 
-enum {
-	kCmpEQ = (1 << 0),
-	kCmpGT = (1 << 1),
-	kCmpLT = (1 << 2)
-};
-
 prcLinkedListStruct *_currentScriptElement;
 byte *_currentScriptPtr;
 uint16 _currentScriptParams;
@@ -982,14 +976,7 @@
 	for (i = 0; i < numZones; i++) {
 		int16 idx;
 
-		// if (gameType == GAME_OS)
-		{
-			idx = getZoneFromPositionRaw(page3Raw, lx + i, ly, 320);
-		}
-		/*  else
-		 * {
-		 * idx = getZoneFromPosition(page3, lx + i, ly, 160);
-		 * } */
+		idx = getZoneFromPositionRaw(page3Raw, lx + i, ly, 320);
 
 		assert(idx >= 0 && idx <= NUM_MAX_ZONE);
 


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.