From ee66c562ab036fd13f89a21a263dff56bc9a2e8d Mon Sep 17 00:00:00 2001 From: JosJuice Date: Mon, 26 Jun 2017 11:14:27 +0200 Subject: [PATCH] TitleDatabase: Let user files override sys files This seems like an oversight in the old code, because what's the point of loading user files if the titles in them are going to be ignored for nearly all games? This commit fixes the issue by making the first LoadMap variant not overwrite entries and making the constructor do everything in the opposite order. An alternative solution would be to make the second LoadMap variant overwrite entries. --- Source/Core/Core/TitleDatabase.cpp | 34 ++++++++++++++++-------------- 1 file changed, 18 insertions(+), 16 deletions(-) diff --git a/Source/Core/Core/TitleDatabase.cpp b/Source/Core/Core/TitleDatabase.cpp index bf5ce2fddf..e041764e01 100644 --- a/Source/Core/Core/TitleDatabase.cpp +++ b/Source/Core/Core/TitleDatabase.cpp @@ -49,6 +49,7 @@ static std::string GetLanguageCode(DiscIO::Language language) using Map = std::unordered_map; +// Note that this function will not overwrite entries that already are in the maps static bool LoadMap(const std::string& file_path, Map& map, std::function predicate) { @@ -69,7 +70,7 @@ static bool LoadMap(const std::string& file_path, Map& map, { const std::string game_id = StripSpaces(line.substr(0, equals_index)); if (game_id.length() >= 4 && predicate(game_id)) - map[game_id] = StripSpaces(line.substr(equals_index + 1)); + map.emplace(game_id, StripSpaces(line.substr(equals_index + 1))); } } return true; @@ -100,6 +101,7 @@ static bool IsNonJapaneseGCTitle(const std::string& game_id) return IsGCTitle(game_id) && DiscIO::RegionSwitchGC(game_id[3]) != DiscIO::Region::NTSC_J; } +// Note that this function will not overwrite entries that already are in the maps static bool LoadMap(const std::string& file_path, Map& gc_map, Map& wii_map) { Map map; @@ -116,17 +118,10 @@ static bool LoadMap(const std::string& file_path, Map& gc_map, Map& wii_map) TitleDatabase::TitleDatabase() { - // Titles that cannot be part of the Wii TDB, - // but common enough to justify having entries for them. - - // i18n: "Wii Menu" (or System Menu) refers to the Wii's main menu, - // which is (usually) the first thing users see when a Wii console starts. - m_wii_title_map.emplace("0000000100000002", GetStringT("Wii Menu")); - for (const auto& id : {"HAXX", "JODI", "00010001af1bf516", "LULZ", "OHBC"}) - m_wii_title_map.emplace(id, "The Homebrew Channel"); - - // Load the English database as the base database. - LoadMap(File::GetSysDirectory() + "wiitdb-en.txt", m_gc_title_map, m_wii_title_map); + // Load the user databases. + const std::string& load_directory = File::GetUserPath(D_LOAD_IDX); + if (!LoadMap(load_directory + "wiitdb.txt", m_gc_title_map, m_wii_title_map)) + LoadMap(load_directory + "titles.txt", m_gc_title_map, m_wii_title_map); // Load the database in the console language. // Note: The GameCube language setting can't be set to Japanese, @@ -142,10 +137,17 @@ TitleDatabase::TitleDatabase() if (wii_code != "en") LoadMap(File::GetSysDirectory() + "wiitdb-" + wii_code + ".txt", m_wii_title_map, IsWiiTitle); - // Load the user databases. - const std::string& load_directory = File::GetUserPath(D_LOAD_IDX); - if (!LoadMap(load_directory + "wiitdb.txt", m_gc_title_map, m_wii_title_map)) - LoadMap(load_directory + "titles.txt", m_gc_title_map, m_wii_title_map); + // Load the English database as the base database. + LoadMap(File::GetSysDirectory() + "wiitdb-en.txt", m_gc_title_map, m_wii_title_map); + + // Titles that cannot be part of the Wii TDB, + // but common enough to justify having entries for them. + + // i18n: "Wii Menu" (or System Menu) refers to the Wii's main menu, + // which is (usually) the first thing users see when a Wii console starts. + m_wii_title_map.emplace("0000000100000002", GetStringT("Wii Menu")); + for (const auto& id : {"HAXX", "JODI", "00010001af1bf516", "LULZ", "OHBC"}) + m_wii_title_map.emplace(id, "The Homebrew Channel"); } TitleDatabase::~TitleDatabase() = default;