diff options
| author | Willem Jan Palenstijn | 2013-08-23 19:15:15 +0200 |
|---|---|---|
| committer | Willem Jan Palenstijn | 2013-08-23 19:15:15 +0200 |
| commit | 97b5114129567e4ca53522f38eb29c1851593d2a (patch) | |
| tree | f5b498fa95a1923791a818c8fd77e927027aede4 | |
| parent | f0c2fa9d65167f03a0aa7879997762c4f3d4702a (diff) | |
| download | scummvm-rg350-97b5114129567e4ca53522f38eb29c1851593d2a.tar.gz scummvm-rg350-97b5114129567e4ca53522f38eb29c1851593d2a.tar.bz2 scummvm-rg350-97b5114129567e4ca53522f38eb29c1851593d2a.zip | |
AGOS: Add two savegame-related FIXMEs
| -rw-r--r-- | engines/agos/saveload.cpp | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/engines/agos/saveload.cpp b/engines/agos/saveload.cpp index e2e0d80c92..8eb7f066b3 100644 --- a/engines/agos/saveload.cpp +++ b/engines/agos/saveload.cpp @@ -33,6 +33,9 @@ namespace AGOS { + +// FIXME: This code counts savegames, but callers in many cases assume +// that the return value + 1 indicates an empty slot. int AGOSEngine::countSaveGames() { Common::InSaveFile *f = NULL; Common::StringArray filenames; @@ -57,6 +60,8 @@ int AGOSEngine::countSaveGames() { marks[slotNum] = true; //mark this slot as valid } + // FIXME: Why does this already try to actually open the savegames? + // Historical accident? while (i < 256) { if (marks[i] && (f = _saveFileMan->openForLoading(genSaveName(i)))) { |