diff options
author | Johannes Schickel | 2011-12-28 01:51:09 +0100 |
---|---|---|
committer | Johannes Schickel | 2011-12-28 01:51:09 +0100 |
commit | db2e0fb0423b575151fde09256c9ccb10ee824e5 (patch) | |
tree | 91ceefaf13ca379f62fc526a2021d2688b64e477 /engines | |
parent | ae8ab8d94e53ce94330987705da2353f7e146ba5 (diff) | |
download | scummvm-rg350-db2e0fb0423b575151fde09256c9ccb10ee824e5.tar.gz scummvm-rg350-db2e0fb0423b575151fde09256c9ccb10ee824e5.tar.bz2 scummvm-rg350-db2e0fb0423b575151fde09256c9ccb10ee824e5.zip |
KYRA: Slight cleanup in the transfer party code.
Diffstat (limited to 'engines')
-rw-r--r-- | engines/kyra/chargen.cpp | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/engines/kyra/chargen.cpp b/engines/kyra/chargen.cpp index 50153d7de3..1093bec079 100644 --- a/engines/kyra/chargen.cpp +++ b/engines/kyra/chargen.cpp @@ -1529,14 +1529,13 @@ bool TransferPartyWiz::start() { } bool TransferPartyWiz::selectAndLoadTransferFile() { - for (int numLoops = 1; numLoops; numLoops--) { + do { _screen->copyPage(12, 0); _vm->_savegameFilename = transferFileDialogue(); - if (_vm->_savegameFilename.empty()) { - if (_vm->_gui->confirmDialogue2(15, 68, 1)) - numLoops++; - } - } + } while (_vm->_savegameFilename.empty() && _vm->_gui->confirmDialogue2(15, 68, 1)); + + if (_vm->_savegameFilename.empty()) + return false; if (_vm->_savegameFilename.equals(_vm->_saveLoadStrings[1])) return false; @@ -1586,7 +1585,7 @@ Common::String TransferPartyWiz::transferFileDialogue() { targets.push_back(*i); } - if (targets.begin() == targets.end()) + if (targets.empty()) return tfile; Common::String target = _vm->_gui->transferTargetMenu(targets); |