Skip to content
Permalink
Browse files
Merge pull request #9717 from JosJuice/android-issettingsaveable
Fix Android controller mappings accidentally being in IsSettingSaveable
  • Loading branch information
lioncash committed May 17, 2021
2 parents 8ecc89f + 1bf6762 commit eae6d91
Showing 1 changed file with 15 additions and 2 deletions.
@@ -26,12 +26,25 @@ bool IsSettingSaveable(const Config::Location& config_location)

if (config_location.system == Config::System::Main)
{
for (const std::string& section : {"NetPlay", "General", "Display", "Network", "Analytics",
"AndroidOverlayButtons", "Android"})
for (const std::string& section :
{"NetPlay", "General", "Display", "Network", "Analytics", "AndroidOverlayButtons"})
{
if (config_location.section == section)
return true;
}

// Android controller mappings are not saveable, other Android settings are.
// TODO: Kill the current Android controller mappings system
if (config_location.section == "Android")
{
static constexpr std::array<const char*, 8> android_setting_saveable = {
"ControlScale", "ControlOpacity", "EmulationOrientation", "JoystickRelCenter",
"LastPlatformTab", "MotionControls", "PhoneRumble", "ShowInputOverlay"};

return std::any_of(
android_setting_saveable.cbegin(), android_setting_saveable.cend(),
[&config_location](const char* key) { return key == config_location.key; });
}
}

static constexpr std::array<const Config::Location*, 17> s_setting_saveable = {

0 comments on commit eae6d91

Please sign in to comment.