[Scummvm-cvs-logs] SF.net SVN: scummvm:[44169] scummvm/trunk/engines/sci/engine

fingolfin at users.sourceforge.net fingolfin at users.sourceforge.net
Thu Sep 17 18:49:31 CEST 2009


Revision: 44169
          http://scummvm.svn.sourceforge.net/scummvm/?rev=44169&view=rev
Author:   fingolfin
Date:     2009-09-17 16:49:31 +0000 (Thu, 17 Sep 2009)

Log Message:
-----------
SCI: Rename WAS_FUNCT_NR to FAKE_FUNCT_NR; turn some #defines into enums

Modified Paths:
--------------
    scummvm/trunk/engines/sci/engine/kernel.h
    scummvm/trunk/engines/sci/engine/kgraphics.cpp
    scummvm/trunk/engines/sci/engine/klists.cpp
    scummvm/trunk/engines/sci/engine/ksound.cpp

Modified: scummvm/trunk/engines/sci/engine/kernel.h
===================================================================
--- scummvm/trunk/engines/sci/engine/kernel.h	2009-09-17 16:40:29 UTC (rev 44168)
+++ scummvm/trunk/engines/sci/engine/kernel.h	2009-09-17 16:49:31 UTC (rev 44169)
@@ -187,10 +187,13 @@
 */
 
 
-enum { WAS_FUNCT_NR = -1 };
+enum {
+	// FIXME: FAKE_FUNCT_NR is a hack used to substitute for an opcode number in certain places
+	FAKE_FUNCT_NR = -1
+};
 
 #define INV_SEL(_object_, _selector_, _noinvalid_) \
-	s, _object_,  ((SciEngine*)g_engine)->getKernel()->_selectorCache._selector_, _noinvalid_, WAS_FUNCT_NR, argv, argc, __FILE__, __LINE__
+	s, _object_,  ((SciEngine*)g_engine)->getKernel()->_selectorCache._selector_, _noinvalid_, FAKE_FUNCT_NR, argv, argc, __FILE__, __LINE__
 /* Kludge for use with invoke_selector(). Used for compatibility with compilers that can't
 ** handle vararg macros.
 */
@@ -298,33 +301,28 @@
 #define SCI_MAX_SAVENAME_LENGTH 0x24
 
 /* Flags for the signal selector */
-#define _K_VIEW_SIG_FLAG_STOP_UPDATE    0x0001
-#define _K_VIEW_SIG_FLAG_UPDATED        0x0002
-#define _K_VIEW_SIG_FLAG_NO_UPDATE      0x0004
-#define _K_VIEW_SIG_FLAG_HIDDEN         0x0008
-#define _K_VIEW_SIG_FLAG_FIX_PRI_ON     0x0010
-#define _K_VIEW_SIG_FLAG_ALWAYS_UPDATE  0x0020
-#define _K_VIEW_SIG_FLAG_FORCE_UPDATE   0x0040
-#define _K_VIEW_SIG_FLAG_REMOVE         0x0080
-#define _K_VIEW_SIG_FLAG_FROZEN         0x0100
-#define _K_VIEW_SIG_FLAG_IS_EXTRA       0x0200
-#define _K_VIEW_SIG_FLAG_HIT_OBSTACLE   0x0400
-#define _K_VIEW_SIG_FLAG_DOESNT_TURN    0x0800
-#define _K_VIEW_SIG_FLAG_NO_CYCLER      0x1000
-#define _K_VIEW_SIG_FLAG_IGNORE_HORIZON 0x2000
-#define _K_VIEW_SIG_FLAG_IGNORE_ACTOR   0x4000
-#define _K_VIEW_SIG_FLAG_DISPOSE_ME     0x8000
+enum {
+	_K_VIEW_SIG_FLAG_STOP_UPDATE    = 0x0001,
+	_K_VIEW_SIG_FLAG_UPDATED        = 0x0002,
+	_K_VIEW_SIG_FLAG_NO_UPDATE      = 0x0004,
+	_K_VIEW_SIG_FLAG_HIDDEN         = 0x0008,
+	_K_VIEW_SIG_FLAG_FIX_PRI_ON     = 0x0010,
+	_K_VIEW_SIG_FLAG_ALWAYS_UPDATE  = 0x0020,
+	_K_VIEW_SIG_FLAG_FORCE_UPDATE   = 0x0040,
+	_K_VIEW_SIG_FLAG_REMOVE         = 0x0080,
+	_K_VIEW_SIG_FLAG_FROZEN         = 0x0100,
+	_K_VIEW_SIG_FLAG_IS_EXTRA       = 0x0200,
+	_K_VIEW_SIG_FLAG_HIT_OBSTACLE   = 0x0400,
+	_K_VIEW_SIG_FLAG_DOESNT_TURN    = 0x0800,
+	_K_VIEW_SIG_FLAG_NO_CYCLER      = 0x1000,
+	_K_VIEW_SIG_FLAG_IGNORE_HORIZON = 0x2000,
+	_K_VIEW_SIG_FLAG_IGNORE_ACTOR   = 0x4000,
+	_K_VIEW_SIG_FLAG_DISPOSE_ME     = 0x8000,
 
-#define _K_VIEW_SIG_FLAG_STOPUPD 0x20000000 /* View has been stop-updated */
+	_K_VIEW_SIG_FLAG_STOPUPD        = 0x20000000 /* View has been stop-updated */
+};
 
 
-/* Sound status */
-#define _K_SOUND_STATUS_STOPPED 0
-#define _K_SOUND_STATUS_INITIALIZED 1
-#define _K_SOUND_STATUS_PAUSED 2
-#define _K_SOUND_STATUS_PLAYING 3
-
-
 /******************** Kernel functions ********************/
 
 // New kernel functions

Modified: scummvm/trunk/engines/sci/engine/kgraphics.cpp
===================================================================
--- scummvm/trunk/engines/sci/engine/kgraphics.cpp	2009-09-17 16:40:29 UTC (rev 44168)
+++ scummvm/trunk/engines/sci/engine/kgraphics.cpp	2009-09-17 16:49:31 UTC (rev 44169)
@@ -343,9 +343,9 @@
 reg_t kSetCursor(EngineState *s, int, int argc, reg_t *argv) {
 	switch (s->detectSetCursorType()) {
 	case SCI_VERSION_0_EARLY:
-		return kSetCursorSci0(s, WAS_FUNCT_NR, argc, argv);
+		return kSetCursorSci0(s, FAKE_FUNCT_NR, argc, argv);
 	case SCI_VERSION_1_1:
-		return kSetCursorSci11(s, WAS_FUNCT_NR, argc, argv);
+		return kSetCursorSci11(s, FAKE_FUNCT_NR, argc, argv);
 	default:
 		warning("Unknown SetCursor type");
 		return NULL_REG;
@@ -855,7 +855,7 @@
 reg_t kCantBeHere(EngineState *s, int, int argc, reg_t *argv) {
 	// kCantBeHere does the same thing as kCanBeHere, except that
 	// it returns the opposite result.
-	reg_t result = kCanBeHere(s, 0, argc, argv);
+	reg_t result = kCanBeHere(s, FAKE_FUNCT_NR, argc, argv);
 	result.offset = !result.offset;
 	return result;
 }
@@ -2992,7 +2992,7 @@
 		// End of doit() recovery code
 
 		if (s->pic_is_new) { // Happens if DrawPic() is executed by a dynview (yes, that happens)
-			kAnimate(s, WAS_FUNCT_NR, argc, argv); /* Tail-recurse */
+			kAnimate(s, FAKE_FUNCT_NR, argc, argv); /* Tail-recurse */
 			return s->r_acc;
 		}
 

Modified: scummvm/trunk/engines/sci/engine/klists.cpp
===================================================================
--- scummvm/trunk/engines/sci/engine/klists.cpp	2009-09-17 16:40:29 UTC (rev 44168)
+++ scummvm/trunk/engines/sci/engine/klists.cpp	2009-09-17 16:49:31 UTC (rev 44169)
@@ -383,7 +383,7 @@
 }
 
 reg_t kDeleteKey(EngineState *s, int, int argc, reg_t *argv) {
-	reg_t node_pos = kFindKey(s, WAS_FUNCT_NR, 2, argv);
+	reg_t node_pos = kFindKey(s, FAKE_FUNCT_NR, 2, argv);
 	Node *n;
 	List *l = lookup_list(s, argv[0]);
 

Modified: scummvm/trunk/engines/sci/engine/ksound.cpp
===================================================================
--- scummvm/trunk/engines/sci/engine/ksound.cpp	2009-09-17 16:40:29 UTC (rev 44168)
+++ scummvm/trunk/engines/sci/engine/ksound.cpp	2009-09-17 16:49:31 UTC (rev 44169)
@@ -36,7 +36,15 @@
 
 namespace Sci {
 
+/* Sound status */
 enum {
+	_K_SOUND_STATUS_STOPPED = 0,
+	_K_SOUND_STATUS_INITIALIZED = 1,
+	_K_SOUND_STATUS_PAUSED = 2,
+	_K_SOUND_STATUS_PLAYING = 3
+};
+
+enum {
 	_K_SCI0_SOUND_INIT_HANDLE = 0,
 	_K_SCI0_SOUND_PLAY_HANDLE = 1,
 	_K_SCI0_SOUND_NOP = 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