diff options
author | Max Horn | 2006-12-20 00:08:51 +0000 |
---|---|---|
committer | Max Horn | 2006-12-20 00:08:51 +0000 |
commit | 6eef02245cb5cf59d89eef8e020ae90b98109483 (patch) | |
tree | dcbda2a08463a6cfa0ee1978c09043dbb900692e /engines/kyra | |
parent | dc30c2304c808dd25920e5f9a179533eb7a45555 (diff) | |
download | scummvm-rg350-6eef02245cb5cf59d89eef8e020ae90b98109483.tar.gz scummvm-rg350-6eef02245cb5cf59d89eef8e020ae90b98109483.tar.bz2 scummvm-rg350-6eef02245cb5cf59d89eef8e020ae90b98109483.zip |
Stripped some more real_ prefixes from function names; also added a new FIXME comment to Kyra's setupGameFlags, and did some more cleanup
svn-id: r24899
Diffstat (limited to 'engines/kyra')
-rw-r--r-- | engines/kyra/plugin.cpp | 34 |
1 files changed, 16 insertions, 18 deletions
diff --git a/engines/kyra/plugin.cpp b/engines/kyra/plugin.cpp index f033f3e800..52d3722905 100644 --- a/engines/kyra/plugin.cpp +++ b/engines/kyra/plugin.cpp @@ -150,21 +150,6 @@ static const KYRAGameDescription adGameDescs[] = { { { NULL, NULL, NULL, UNK_LANG, kPlatformUnknown }, KYRA2_UNK_FLAGS } }; -static ADList detectKyraGames(const FSList &fslist) { - Common::AdvancedDetector ad; - Common::ADList matches; - Common::ADGameDescList descList; - - for (int i = 0; i < ARRAYSIZE(adGameDescs) - 1; ++i) { - descList.push_back(&adGameDescs[i].desc); - } - - ad.registerGameDescriptions(descList); - - matches = ad.detectGame(&fslist, kMD5FileSizeLimit, Common::UNK_LANG, Common::kPlatformUnknown); - return matches; -} - static bool setupGameFlags(const ADList &list, GameFlags &flags) { if (!list.size()) { // maybe add non md5 based detection again? @@ -173,6 +158,8 @@ static bool setupGameFlags(const ADList &list, GameFlags &flags) { int id = list[0]; + // FIXME: Isn't the following check/loop obsolete (i.e. I was/am under the + // impression that AdvancedDetector already performs this check). if (list.size() > 1) { int filesCount = 0; int curID = 0; @@ -222,7 +209,7 @@ GameList Engine_KYRA_gameIDList() { } GameDescriptor Engine_KYRA_findGameID(const char *gameid) { - return Common::real_ADVANCED_DETECTOR_FIND_GAMEID(gameid, gameList, 0); + return Common::ADVANCED_DETECTOR_FIND_GAMEID(gameid, gameList, 0); } DetectedGameList Engine_KYRA_detectGames(const FSList &fslist) { @@ -246,8 +233,19 @@ PluginError Engine_KYRA_create(OSystem *syst, Engine **engine) { } GameFlags flags; - ADList games = detectKyraGames(fslist); - if (!setupGameFlags(games, flags)) { + Common::AdvancedDetector ad; + Common::ADList matches; + Common::ADGameDescList descList; + + for (int i = 0; i < ARRAYSIZE(adGameDescs) - 1; ++i) { + descList.push_back(&adGameDescs[i].desc); + } + + ad.registerGameDescriptions(descList); + + matches = ad.detectGame(&fslist, kMD5FileSizeLimit, Common::UNK_LANG, Common::kPlatformUnknown); + + if (!setupGameFlags(matches, flags)) { return kNoGameDataFoundError; } |