[Scummvm-cvs-logs] SF.net SVN: scummvm:[39882] scummvm/trunk/engines/gob

fingolfin at users.sourceforge.net fingolfin at users.sourceforge.net
Tue Apr 7 12:41:23 CEST 2009


Revision: 39882
          http://scummvm.svn.sourceforge.net/scummvm/?rev=39882&view=rev
Author:   fingolfin
Date:     2009-04-07 10:41:23 +0000 (Tue, 07 Apr 2009)

Log Message:
-----------
GOB: Gave some numerical constants a name. This codebase is a bit of a mess ;) (tons of duplicate code, unnamed constants, etc.)

Modified Paths:
--------------
    scummvm/trunk/engines/gob/goblin_v1.cpp
    scummvm/trunk/engines/gob/parse.cpp
    scummvm/trunk/engines/gob/parse.h
    scummvm/trunk/engines/gob/parse_v1.cpp
    scummvm/trunk/engines/gob/parse_v2.cpp

Modified: scummvm/trunk/engines/gob/goblin_v1.cpp
===================================================================
--- scummvm/trunk/engines/gob/goblin_v1.cpp	2009-04-07 05:36:12 UTC (rev 39881)
+++ scummvm/trunk/engines/gob/goblin_v1.cpp	2009-04-07 10:41:23 UTC (rev 39882)
@@ -671,6 +671,7 @@
 				_gobPositions[_currentGoblin].y++;
 				break;
 			}
+
 			if (_currentGoblin == 1) {
 				if (gobDesc->nextState == 28)
 					_gobPositions[1].y--;

Modified: scummvm/trunk/engines/gob/parse.cpp
===================================================================
--- scummvm/trunk/engines/gob/parse.cpp	2009-04-07 05:36:12 UTC (rev 39881)
+++ scummvm/trunk/engines/gob/parse.cpp	2009-04-07 10:41:23 UTC (rev 39882)
@@ -304,31 +304,31 @@
 			debugN(5, "-");
 			break;
 
-		case 2:
+		case OP_ADD:
 			debugN(5, "+");
 			break;
 
-		case 3:
+		case OP_SUB:
 			debugN(5, "-");
 			break;
 
-		case 4:
+		case OP_BITOR:
 			debugN(5, "|");
 			break;
 
-		case 5:
+		case OP_MUL:
 			debugN(5, "*");
 			break;
 
-		case 6:
+		case OP_DIV:
 			debugN(5, "/");
 			break;
 
-		case 7:
+		case OP_MOD:
 			debugN(5, "%%");
 			break;
 
-		case 8:
+		case OP_BITAND:
 			debugN(5, "&");
 			break;
 

Modified: scummvm/trunk/engines/gob/parse.h
===================================================================
--- scummvm/trunk/engines/gob/parse.h	2009-04-07 05:36:12 UTC (rev 39881)
+++ scummvm/trunk/engines/gob/parse.h	2009-04-07 10:41:23 UTC (rev 39882)
@@ -28,6 +28,16 @@
 
 namespace Gob {
 
+enum {
+	OP_ADD = 2,
+	OP_SUB = 3,
+	OP_BITOR = 4,
+	OP_MUL = 5,
+	OP_DIV = 6,
+	OP_MOD = 7,
+	OP_BITAND = 8
+};
+
 class Parse {
 public:
 	void skipExpr(char stopToken);

Modified: scummvm/trunk/engines/gob/parse_v1.cpp
===================================================================
--- scummvm/trunk/engines/gob/parse_v1.cpp	2009-04-07 05:36:12 UTC (rev 39881)
+++ scummvm/trunk/engines/gob/parse_v1.cpp	2009-04-07 10:41:23 UTC (rev 39882)
@@ -197,19 +197,19 @@
 				valPtr -= 2;
 
 				switch (operPtr[1]) {
-				case 5:
+				case OP_MUL:
 					valPtr[0] *= valPtr[2];
 					break;
 
-				case 6:
+				case OP_DIV:
 					valPtr[0] /= valPtr[2];
 					break;
 
-				case 7:
+				case OP_MOD:
 					valPtr[0] %= valPtr[2];
 					break;
 
-				case 8:
+				case OP_BITAND:
 					valPtr[0] &= valPtr[2];
 					break;
 				}
@@ -244,19 +244,19 @@
 					operPtr -= 2;
 					valPtr -= 2;
 					switch (operPtr[0]) {
-					case 5:
+					case OP_MUL:
 						operPtr[-1] *= operPtr[1];
 						break;
 
-					case 6:
+					case OP_DIV:
 						operPtr[-1] /= operPtr[1];
 						break;
 
-					case 7:
+					case OP_MOD:
 						operPtr[-1] %= operPtr[1];
 						break;
 
-					case 8:
+					case OP_BITAND:
 						operPtr[-1] &= operPtr[1];
 						break;
 					}
@@ -280,25 +280,25 @@
 			operPtr -= 2;
 			valPtr -= 2;
 			switch (operPtr[0]) {
-			case 2:
+			case OP_ADD:
 				values[brackPos] += valPtr[1];
 				continue;
-			case 3:
+			case OP_SUB:
 				values[brackPos] -= valPtr[1];
 				continue;
-			case 4:
+			case OP_BITOR:
 				values[brackPos] |= valPtr[1];
 				continue;
-			case 5:
+			case OP_MUL:
 				valPtr[-1] *= valPtr[1];
 				continue;
-			case 6:
+			case OP_DIV:
 				valPtr[-1] /= valPtr[1];
 				continue;
-			case 7:
+			case OP_MOD:
 				valPtr[-1] %= valPtr[1];
 				continue;
-			case 8:
+			case OP_BITAND:
 				valPtr[-1] &= valPtr[1];
 				continue;
 			}
@@ -487,28 +487,28 @@
 				}
 				break;
 
-			case 5:
+			case OP_MUL:
 				valPtr[-2] *= valPtr[0];
 				stkPos -= 2;
 				operPtr -= 2;
 				valPtr -= 2;
 				break;
 
-			case 6:
+			case OP_DIV:
 				valPtr[-2] /= valPtr[0];
 				stkPos -= 2;
 				operPtr -= 2;
 				valPtr -= 2;
 				break;
 
-			case 7:
+			case OP_MOD:
 				valPtr[-2] %= valPtr[0];
 				stkPos -= 2;
 				operPtr -= 2;
 				valPtr -= 2;
 				break;
 
-			case 8:
+			case OP_BITAND:
 				valPtr[-2] &= valPtr[0];
 				stkPos -= 2;
 				operPtr -= 2;
@@ -553,27 +553,27 @@
 
 					if (stkPos > 2) {
 						switch (operPtr[-2]) {
-						case 5:
+						case OP_MUL:
 							valPtr[-3] *= valPtr[-1];
 							stkPos -= 2;
 							operPtr -= 2;
 							valPtr -= 2;
 							break;
-						case 6:
+						case OP_DIV:
 							valPtr[-3] /= valPtr[-1];
 							stkPos -= 2;
 							operPtr -= 2;
 							valPtr -= 2;
 							break;
 
-						case 7:
+						case OP_MOD:
 							valPtr[-3] %= valPtr[-1];
 							stkPos -= 2;
 							operPtr -= 2;
 							valPtr -= 2;
 							break;
 
-						case 8:
+						case OP_BITAND:
 							valPtr[-3] &= valPtr[-1];
 							stkPos -= 2;
 							operPtr -= 2;
@@ -611,42 +611,42 @@
 					valPtr -= 2;
 					continue;
 
-				case 3:
+				case OP_SUB:
 					values[brackStart] -= valPtr[-1];
 					stkPos -= 2;
 					operPtr -= 2;
 					valPtr -= 2;
 					continue;
 
-				case 4:
+				case OP_BITOR:
 					values[brackStart] |= valPtr[-1];
 					stkPos -= 2;
 					operPtr -= 2;
 					valPtr -= 2;
 					continue;
 
-				case 5:
+				case OP_MUL:
 					valPtr[-3] *= valPtr[-1];
 					stkPos -= 2;
 					operPtr -= 2;
 					valPtr -= 2;
 					break;
 
-				case 6:
+				case OP_DIV:
 					valPtr[-3] /= valPtr[-1];
 					stkPos -= 2;
 					operPtr -= 2;
 					valPtr -= 2;
 					break;
 
-				case 7:
+				case OP_MOD:
 					valPtr[-3] %= valPtr[-1];
 					stkPos -= 2;
 					operPtr -= 2;
 					valPtr -= 2;
 					break;
 
-				case 8:
+				case OP_BITAND:
 					valPtr[-3] &= valPtr[-1];
 					stkPos -= 2;
 					operPtr -= 2;

Modified: scummvm/trunk/engines/gob/parse_v2.cpp
===================================================================
--- scummvm/trunk/engines/gob/parse_v2.cpp	2009-04-07 05:36:12 UTC (rev 39881)
+++ scummvm/trunk/engines/gob/parse_v2.cpp	2009-04-07 10:41:23 UTC (rev 39882)
@@ -335,19 +335,19 @@
 				valPtr -= 2;
 
 				switch (operPtr[1]) {
-				case 5:
+				case OP_MUL:
 					valPtr[0] *= valPtr[2];
 					break;
 
-				case 6:
+				case OP_DIV:
 					valPtr[0] /= valPtr[2];
 					break;
 
-				case 7:
+				case OP_MOD:
 					valPtr[0] %= valPtr[2];
 					break;
 
-				case 8:
+				case OP_BITAND:
 					valPtr[0] &= valPtr[2];
 					break;
 				}
@@ -383,19 +383,19 @@
 					operPtr -= 2;
 					valPtr -= 2;
 					switch (operPtr[0]) {
-					case 5:
+					case OP_MUL:
 						valPtr[-1] *= valPtr[1];
 						break;
 
-					case 6:
+					case OP_DIV:
 						valPtr[-1] /= valPtr[1];
 						break;
 
-					case 7:
+					case OP_MOD:
 						valPtr[-1] %= valPtr[1];
 						break;
 
-					case 8:
+					case OP_BITAND:
 						valPtr[-1] &= valPtr[1];
 						break;
 					}
@@ -419,25 +419,25 @@
 			operPtr -= 2;
 			valPtr -= 2;
 			switch (operPtr[0]) {
-			case 2:
+			case OP_ADD:
 				values[brackPos] += valPtr[1];
 				continue;
-			case 3:
+			case OP_SUB:
 				values[brackPos] -= valPtr[1];
 				continue;
-			case 4:
+			case OP_BITOR:
 				values[brackPos] |= valPtr[1];
 				continue;
-			case 5:
+			case OP_MUL:
 				valPtr[-1] *= valPtr[1];
 				continue;
-			case 6:
+			case OP_DIV:
 				valPtr[-1] /= valPtr[1];
 				continue;
-			case 7:
+			case OP_MOD:
 				valPtr[-1] %= valPtr[1];
 				continue;
-			case 8:
+			case OP_BITAND:
 				valPtr[-1] &= valPtr[1];
 				continue;
 			}
@@ -693,28 +693,28 @@
 				}
 				break;
 
-			case 5:
+			case OP_MUL:
 				valPtr[-2] *= valPtr[0];
 				stkPos -= 2;
 				operPtr -= 2;
 				valPtr -= 2;
 				break;
 
-			case 6:
+			case OP_DIV:
 				valPtr[-2] /= valPtr[0];
 				stkPos -= 2;
 				operPtr -= 2;
 				valPtr -= 2;
 				break;
 
-			case 7:
+			case OP_MOD:
 				valPtr[-2] %= valPtr[0];
 				stkPos -= 2;
 				operPtr -= 2;
 				valPtr -= 2;
 				break;
 
-			case 8:
+			case OP_BITAND:
 				valPtr[-2] &= valPtr[0];
 				stkPos -= 2;
 				operPtr -= 2;
@@ -756,28 +756,28 @@
 
 					if (stkPos > 2) {
 						switch (operPtr[-2]) {
-						case 5:
+						case OP_MUL:
 							valPtr[-3] *= valPtr[-1];
 							stkPos -= 2;
 							operPtr -= 2;
 							valPtr -= 2;
 							break;
 
-						case 6:
+						case OP_DIV:
 							valPtr[-3] /= valPtr[-1];
 							stkPos -= 2;
 							operPtr -= 2;
 							valPtr -= 2;
 							break;
 
-						case 7:
+						case OP_MOD:
 							valPtr[-3] %= valPtr[-1];
 							stkPos -= 2;
 							operPtr -= 2;
 							valPtr -= 2;
 							break;
 
-						case 8:
+						case OP_BITAND:
 							valPtr[-3] &= valPtr[-1];
 							stkPos -= 2;
 							operPtr -= 2;
@@ -815,42 +815,42 @@
 					valPtr -= 2;
 					continue;
 
-				case 3:
+				case OP_SUB:
 					values[brackStart] -= valPtr[-1];
 					stkPos -= 2;
 					operPtr -= 2;
 					valPtr -= 2;
 					continue;
 
-				case 4:
+				case OP_BITOR:
 					values[brackStart] |= valPtr[-1];
 					stkPos -= 2;
 					operPtr -= 2;
 					valPtr -= 2;
 					continue;
 
-				case 5:
+				case OP_MUL:
 					valPtr[-3] *= valPtr[-1];
 					stkPos -= 2;
 					operPtr -= 2;
 					valPtr -= 2;
 					break;
 
-				case 6:
+				case OP_DIV:
 					valPtr[-3] /= valPtr[-1];
 					stkPos -= 2;
 					operPtr -= 2;
 					valPtr -= 2;
 					break;
 
-				case 7:
+				case OP_MOD:
 					valPtr[-3] %= valPtr[-1];
 					stkPos -= 2;
 					operPtr -= 2;
 					valPtr -= 2;
 					break;
 
-				case 8:
+				case OP_BITAND:
 					valPtr[-3] &= valPtr[-1];
 					stkPos -= 2;
 					operPtr -= 2;


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




More information about the Scummvm-git-logs mailing list