Skip to content

Commit

Permalink
UICommon: Remove Android hacks from GameFile
Browse files Browse the repository at this point in the history
  • Loading branch information
JosJuice committed Aug 21, 2019
1 parent c677268 commit f79ca65
Showing 1 changed file with 3 additions and 20 deletions.
23 changes: 3 additions & 20 deletions Source/Core/UICommon/GameFile.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -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)
Expand Down Expand Up @@ -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;
Expand All @@ -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;
Expand Down Expand Up @@ -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;
Expand Down

0 comments on commit f79ca65

Please sign in to comment.