libretro/scummvm
1
0
Fork 0
mirror of https://github.com/libretro/scummvm.git synced 2025-01-19 00:15:30 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Remove obsolete and unneeded logic bypassing reloading a location.

Browse Source 
I have tested that this could only possibly happen when the game has been
loaded with last location being the map.  Then pressing Escape calls
enterNewRoom() and this superfluous optimization takes place.  It is harmless
to simply reload the map.  After having removed it, enterNewRoom() needs not
return any return value, because the test at the tail can be done by the
caller.  I have then restructured the code a little to make it cleaner.

svn-id: r46098
This commit is contained in:
Robert Špalek 2009-11-22 21:12:23 +00:00
parent d3d16422e0
commit f6f8b66d45
2 changed files with 20 additions and 34 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

52
engines/draci/game.cpp
View File

@ -154,10 +154,21 @@ Game::Game(DraciEngine *vm) : _vm(vm), _walkingState(vm) {
void Game::start() {
while (!shouldQuit()) {
if (enterNewRoom()) {
// Call the outer loop doing all the hard job.
loop(kOuterLoop, false);
// Reset the flag allowing to run the scripts.
_vm->_script->endCurrentProgram(false);
enterNewRoom();
if (_vm->_script->shouldEndProgram()) {
// Escape pressed during the intro or map animations run in the
// init scripts. This flag was turned on to skip the rest of
// those programs. Don't call loop(), because the
// location may have changed.
continue;
}
// Call the outer loop doing all the hard job.
loop(kOuterLoop, false);
}
}
@ -432,6 +443,7 @@ void Game::advanceAnimationsAndTestLoopExit() {
// by the user clicking on something or run at the end of a
// gate script in the intro).
if ((_loopStatus == kStatusOrdinary || _loopStatus == kStatusGate) && (_newRoom != getRoomNum() || _newGate != _variables[0] - 1)) {
// TODO: don't use _variables but a new named attribute
setExitLoop(true);
}
@ -1221,20 +1233,7 @@ void Game::deleteObjectAnimations() {
}
}
bool Game::enterNewRoom() {
if ((_newRoom == getRoomNum() && _newGate == _variables[0] - 1) && !isReloaded()) {
// Re-entering the same room via a different gate is correct
// and the room needs to be reloaded (used in the ballroom when
// propping the chair by the brick).
// TODO: 1. don't use _variables but a new named attribute. 2.
// investigate whether the optimization with shortcut is
// needed; maybe reloading the room always is the right thing
// to do.
_vm->_script->endCurrentProgram(false);
return true;
}
// If the game has been reloaded, force reloading all animations.
setIsReloaded(false);
void Game::enterNewRoom() {
debugC(1, kDraciLogicDebugLevel, "Entering room %d using gate %d", _newRoom, _newGate);
_vm->_mouse->cursorOff();
@ -1285,10 +1284,6 @@ bool Game::enterNewRoom() {
// cleaned up. Some rooms (e.g., the map) don't support walking.
_walkingState.stopWalking();
// Reset the flag allowing to run the scripts. It may have been turned
// on by pressing Escape in the intro or in the map room.
_vm->_script->endCurrentProgram(false);
_currentRoom.load(_newRoom, _vm->_roomsArchive);
loadWalkingMap(getMapID());
loadRoomObjects();
@ -1303,7 +1298,10 @@ bool Game::enterNewRoom() {
debugC(6, kDraciLogicDebugLevel, "Running program for gate %d", _newGate);
_vm->_script->runWrapper(_currentRoom._program, _currentRoom._gates[_newGate], true, true);
// Reset the loop status.
setLoopStatus(kStatusOrdinary);
setExitLoop(false);
setIsReloaded(false);
// Set cursor state
// Need to do this after we set the palette since the cursors use it
@ -1315,18 +1313,6 @@ bool Game::enterNewRoom() {
debugC(6, kDraciLogicDebugLevel, "Mouse: OFF");
_vm->_mouse->cursorOff();
}
// Reset the loop status.
setLoopStatus(kStatusOrdinary);
if (_vm->_script->shouldEndProgram()) {
// Escape pressed during the intro or map animations run in the
// init scripts. This flag was turned on to skip the rest of
// those programs. Return false to make start() rerun us from
// the beginning, because the room number has changed.
return false;
}
return true;
}
void Game::positionAnimAsHero(Animation *anim) {

2
engines/draci/game.h
View File

@ -327,7 +327,7 @@ private:
void advanceAnimationsAndTestLoopExit();
void handleStatusChangeByMouse();
bool enterNewRoom(); // Returns false if another room change has been triggered and therefore loop() shouldn't be called yet.
void enterNewRoom();
void initWalkingOverlays();
void loadRoomObjects();
void redrawWalkingPath(Animation *anim, byte colour, const WalkingPath &path);