From f79ca651701fdbfe1a95aa09bd7c3d0a68ed4eeb Mon Sep 17 00:00:00 2001 From: JosJuice Date: Sun, 16 Jun 2019 16:38:26 +0200 Subject: [PATCH] UICommon: Remove Android hacks from GameFile --- Source/Core/UICommon/GameFile.cpp | 23 +++-------------------- 1 file changed, 3 insertions(+), 20 deletions(-) diff --git a/Source/Core/UICommon/GameFile.cpp b/Source/Core/UICommon/GameFile.cpp index 89d38e24f5..f5aefd7f01 100644 --- a/Source/Core/UICommon/GameFile.cpp +++ b/Source/Core/UICommon/GameFile.cpp @@ -46,28 +46,11 @@ namespace constexpr char COVER_URL[] = "https://art.gametdb.com/wii/cover/%s/%s.png"; const std::string EMPTY_STRING; - -bool UseGameCovers() -{ -// We ifdef this out on Android because accessing the config before emulation start makes us crash. -// The Android GUI handles covers in Java anyway, so this doesn't make us lose any functionality. -#ifdef ANDROID - return false; -#else - return Config::Get(Config::MAIN_USE_GAME_COVERS); -#endif -} } // Anonymous namespace DiscIO::Language GameFile::GetConfigLanguage() const { -#ifdef ANDROID - // TODO: Make the Android app load the config at app start instead of emulation start - // so that we can access the user's preference here - return DiscIO::Language::English; -#else return SConfig::GetInstance().GetLanguageAdjustedForRegion(DiscIO::IsWii(m_platform), m_region); -#endif } bool operator==(const GameBanner& lhs, const GameBanner& rhs) @@ -174,7 +157,7 @@ bool GameFile::IsValid() const bool GameFile::CustomCoverChanged() { - if (!m_custom_cover.buffer.empty() || !UseGameCovers()) + if (!m_custom_cover.buffer.empty() || !Config::Get(Config::MAIN_USE_GAME_COVERS)) return false; std::string path, name; @@ -201,7 +184,7 @@ bool GameFile::CustomCoverChanged() void GameFile::DownloadDefaultCover() { - if (!m_default_cover.buffer.empty() || !UseGameCovers()) + if (!m_default_cover.buffer.empty() || !Config::Get(Config::MAIN_USE_GAME_COVERS)) return; const auto cover_path = File::GetUserPath(D_COVERCACHE_IDX) + DIR_SEP; @@ -267,7 +250,7 @@ void GameFile::DownloadDefaultCover() bool GameFile::DefaultCoverChanged() { - if (!m_default_cover.buffer.empty() || !UseGameCovers()) + if (!m_default_cover.buffer.empty() || !Config::Get(Config::MAIN_USE_GAME_COVERS)) return false; const auto cover_path = File::GetUserPath(D_COVERCACHE_IDX) + DIR_SEP;