[Scummvm-cvs-logs] SF.net SVN: scummvm: [29585] scummvm/trunk/engines/agi

thebluegr at users.sourceforge.net thebluegr at users.sourceforge.net
Tue Nov 20 21:25:37 CET 2007


Revision: 29585
          http://scummvm.svn.sourceforge.net/scummvm/?rev=29585&view=rev
Author:   thebluegr
Date:     2007-11-20 12:25:37 -0800 (Tue, 20 Nov 2007)

Log Message:
-----------
Cleanup and simplification

Modified Paths:
--------------
    scummvm/trunk/engines/agi/preagi_mickey.cpp
    scummvm/trunk/engines/agi/preagi_mickey.h

Modified: scummvm/trunk/engines/agi/preagi_mickey.cpp
===================================================================
--- scummvm/trunk/engines/agi/preagi_mickey.cpp	2007-11-20 19:59:29 UTC (rev 29584)
+++ scummvm/trunk/engines/agi/preagi_mickey.cpp	2007-11-20 20:25:37 UTC (rev 29585)
@@ -104,12 +104,8 @@
 bool Mickey::chooseY_N(int ofsPrompt, bool fErrorMsg) {
 	printExeStr(ofsPrompt);
 
-	_vm->_gfx->doUpdate();
-	_vm->_system->updateScreen();	// TODO: this should go in the game's main loop
-
-	int a = _vm->getSelection(kSelYesNo);
 	for (;;) {
-		switch (a) {
+		switch (_vm->getSelection(kSelYesNo)) {
 			case 0: return false;
 			case 1: return true;
 			default: if (fErrorMsg) {
@@ -119,25 +115,21 @@
 					}
 					break;
 		}
-		a = _vm->getSelection(kSelYesNo);
 	}
 }
 
 int Mickey::choose1to9(int ofsPrompt) {
+	int answer = 0;
 	printExeStr(ofsPrompt);
 
-	_vm->_gfx->doUpdate();
-	_vm->_system->updateScreen();	// TODO: this should go in the game's main loop
-
-	int a = _vm->getSelection(kSelNumber);
 	for (;;) {
-		if (a == 10) {
+		answer = _vm->getSelection(kSelNumber);
+		if (answer == 10) {
 			printExeStr(IDO_MSA_PRESS_1_TO_9);
 			if (_vm->getSelection(kSelAnyKey) == 0)
 				return 0;
 			printExeStr(ofsPrompt);
-		} else return a;
-		a = _vm->getSelection(kSelNumber);
+		} else return answer;
 	}
 
 }
@@ -155,6 +147,10 @@
 		_vm->drawStr(iRow, iCol, IDA_DEFAULT, buffer + pc);
 		pc += strlen(buffer + pc) + 1;
 	}
+
+	// Show the string on screen
+	_vm->_gfx->doUpdate();
+	_vm->_system->updateScreen();
 }
 
 void Mickey::printExeStr(int ofs) {
@@ -165,8 +161,6 @@
 
 	readExe(ofs, buffer, sizeof(buffer));
 	printStr((char *)buffer);
-	_vm->_gfx->doUpdate();
-	_vm->_system->updateScreen();	// TODO: this should go in the game's main loop
 }
 
 void Mickey::printExeMsg(int ofs) {
@@ -174,7 +168,7 @@
 		return;
 
 	printExeStr(ofs);
-	waitAnyKeyAnim();
+	waitAnyKey(true);
 }
 
 void Mickey::printDatString(int iStr) {
@@ -221,8 +215,6 @@
 	infile.close();
 
 	printStr(buffer);
-	//_vm->_gfx->doUpdate();
-	//_vm->_system->updateScreen();	// TODO: this should go in the game's main loop
 	free(buffer);
 }
 
@@ -252,9 +244,6 @@
 							 attr, (char *)menu.row[iRow].entry[iWord].szText);
 		}
 	}
-
-	_vm->_gfx->doUpdate();
-	_vm->_system->updateScreen();	// TODO: this should go in the game's main loop
 }
 
 void Mickey::getMouseMenuSelRow(MSA_MENU menu, int *sel0, int *sel1, int iRow, int x, int y) {
@@ -414,8 +403,6 @@
 							printExeMsg(IDO_MSA_HIDDEN_MSG[i]);
 						}
 						_vm->clearTextArea();
-						//_vm->_gfx->doUpdate();
-						//_vm->_system->updateScreen();	// TODO: this should go in the game's main loop
 						waitAnyKey();
 					}
 					break;
@@ -572,9 +559,7 @@
 
 void Mickey::printDatMessage(int iStr) {
 	printDatString(iStr);
-	//_vm->_gfx->doUpdate();
-	//_vm->_system->updateScreen();	// TODO: this should go in the game's main loop
-	waitAnyKeyAnim();
+	waitAnyKey(true);
 }
 
 // Sound
@@ -924,7 +909,7 @@
 void Mickey::printRoomDesc() {
 	// print room description
 	printDesc(_game.iRoom);
-	waitAnyKeyAnim();
+	waitAnyKey(true);
 
 	// print extended room description
 	if (_game.oRmTxt[_game.iRoom]) {
@@ -1026,8 +1011,6 @@
 		_vm->drawStr(iRow, 0, IDA_DEFAULT, szLine);
 		pBuf += strlen(szLine) + 1;
 	}
-	//_vm->_gfx->doUpdate();
-	//_vm->_system->updateScreen();	// TODO: this should go in the game's main loop
 	waitAnyKey();
 
 	_vm->clearScreen(IDA_DEFAULT);
@@ -1036,8 +1019,6 @@
 		_vm->drawStr(iRow, 0, IDA_DEFAULT, szLine);
 		pBuf += strlen(szLine) + 1;
 	}
-	//_vm->_gfx->doUpdate();
-	//_vm->_system->updateScreen();	// TODO: this should go in the game's main loop
 	waitAnyKey();
 
 	//Set back to black
@@ -1089,16 +1070,12 @@
 	// print pressed buttons
 	printExeStr(IDO_MSA_MICKEY_HAS_PRESSED);
 	_vm->drawStr(IDI_MSA_ROW_BUTTONS, IDI_MSA_COL_BUTTONS, IDA_DEFAULT, szButtons);
-	//_vm->_gfx->doUpdate();
-	//_vm->_system->updateScreen();	// TODO: this should go in the game's main loop
 	waitAnyKey();
 }
 
 void Mickey::insertDisk(int iDisk) {
 	_vm->clearTextArea();
 	_vm->drawStr(IDI_MSA_ROW_INSERT_DISK, IDI_MSA_COL_INSERT_DISK, IDA_DEFAULT, (const char *)IDS_MSA_INSERT_DISK[iDisk]);
-	//_vm->_gfx->doUpdate();
-	//_vm->_system->updateScreen();	// TODO: this should go in the game's main loop
 	waitAnyKey();
 }
 
@@ -1154,8 +1131,6 @@
 		_game.fAnimXL30 = true;
 
 		_vm->clearTextArea();
-		//_vm->_gfx->doUpdate();
-		//_vm->_system->updateScreen();	// TODO: this should go in the game's main loop
 		playSound(IDI_MSA_SND_XL30);
 		printExeMsg(IDO_MSA_XL30_SPEAKING);
 
@@ -1188,7 +1163,7 @@
 			_vm->_system->updateScreen();	// TODO: this should go in the game's main loop
 #endif
 
-			waitAnyKeyAnim();
+			waitAnyKey(true);
 		}
 	} else {
 		printStory();
@@ -1213,8 +1188,6 @@
 		}
 	}
 
-	//_vm->_gfx->doUpdate();
-	//_vm->_system->updateScreen();	// TODO: this should go in the game's main loop
 	waitAnyKey();
 
 	_vm->clearScreen(IDA_DEFAULT);
@@ -1531,8 +1504,6 @@
 		if (_game.iRmPic[_game.iRoom] == IDI_MSA_PIC_VENUS_PROBE_1) {
 			_vm->clearRow(22);
 		}
-		//_vm->_gfx->doUpdate();
-		//_vm->_system->updateScreen();	// TODO: this should go in the game's main loop
 		waitAnyKey();
 		break;
 	case IDI_MSA_ACTION_GET_XTAL_VENUS:
@@ -1867,8 +1838,6 @@
 			(const char *)IDS_MSA_TEMP_C[_game.iPlanet]);
 		_vm->drawStr(IDI_MSA_ROW_TEMPERATURE, IDI_MSA_COL_TEMPERATURE_F, IDA_DEFAULT,
 			(const char *)IDS_MSA_TEMP_F[_game.iPlanet]);
-		//_vm->_gfx->doUpdate();
-		//_vm->_system->updateScreen();	// TODO: this should go in the game's main loop
 		waitAnyKey();
 		break;
 	case IDI_MSA_ACTION_PRESS_ORANGE:
@@ -1901,9 +1870,7 @@
 			printDatString(22);
 			_vm->drawStr(IDI_MSA_ROW_PLANET, IDI_MSA_COL_PLANET, IDA_DEFAULT,
 						(const char *)IDS_MSA_PLANETS[_game.iPlanet]);
-			//_vm->_gfx->doUpdate();
-			//_vm->_system->updateScreen();	// TODO: this should go in the game's main loop
-			waitAnyKeyAnim();
+			waitAnyKey(true);
 			showPlanetInfo();
 		} else {
 			printDatMessage(arg);
@@ -1964,10 +1931,6 @@
 
 // Keyboard
 
-void Mickey::waitAnyKeyAnim() {
-	waitAnyKey(true);
-}
-
 void Mickey::waitAnyKey(bool anim) {
 	Common::Event event;
 
@@ -2010,8 +1973,6 @@
 		sprintf(szTitle, "Object %d", iObj);
 		_vm->drawStrMiddle(22, IDA_DEFAULT, szTitle);
 		_vm->drawStrMiddle(23, IDA_DEFAULT, (const char *)IDS_MSA_NAME_OBJ[iObj]);
-		//_vm->_gfx->doUpdate();
-		//_vm->_system->updateScreen();	// TODO: this should go in the game's main loop
 		waitAnyKey();
 	}
 }
@@ -2025,8 +1986,6 @@
 		_vm->clearTextArea();
 		sprintf(szTitle, "Picture %d", iPic);
 		_vm->drawStrMiddle(22, IDA_DEFAULT, szTitle);
-		//_vm->_gfx->doUpdate();
-		//_vm->_system->updateScreen();	// TODO: this should go in the game's main loop
 		waitAnyKey();
 	}
 }

Modified: scummvm/trunk/engines/agi/preagi_mickey.h
===================================================================
--- scummvm/trunk/engines/agi/preagi_mickey.h	2007-11-20 19:59:29 UTC (rev 29584)
+++ scummvm/trunk/engines/agi/preagi_mickey.h	2007-11-20 20:25:37 UTC (rev 29585)
@@ -787,7 +787,6 @@
 	void debug_DrawObjs();
 	void debug_DrawPics();
 	void flipSwitch();
-	void waitAnyKeyAnim();
 	void waitAnyKey(bool anim = false);
 
 	bool planetIsAlreadyAssigned(int planet) {


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