Clean up accessing system setting values for a specific player.

Remove the ugly "save, change and restore platform" accesses and extend the settings class to allow accessing player specific system values.

git-svn-id: svn://svn.rockbox.org/rockbox/trunk@22254 a1c6a512-1295-4272-9138-f99709370657

1 files changed, 20 insertions, 0 deletions

diff --git a/rbutil/rbutilqt/base/rbsettings.cpp b/rbutil/rbutilqt/base/rbsettings.cpp
index 8ab340faa3..150640f010 100644
--- a/rbutil/rbutilqt/base/rbsettings.cpp
+++ b/rbutil/rbutilqt/base/rbsettings.cpp
@@ -220,6 +220,25 @@ void RbSettings::setSubValue(QString sub, enum UserSettings setting, QVariant va
     userSettings->setValue(s, value);
 }
 
+
+QVariant RbSettings::platformValue(QString platform, enum SystemSettings setting)
+{
+    ensureRbSettingsExists();
+
+    // locate setting item
+    int i = 0;
+    while(SystemSettingsList[i].setting != setting)
+        i++;
+
+    QString s = SystemSettingsList[i].name;
+    s.replace(":platform:", platform);
+    QString d = SystemSettingsList[i].def;
+    d.replace(":platform:", platform);
+    qDebug() << "[Settings] GET P:" << s << systemSettings->value(s, d).toString();
+    return systemSettings->value(s, d);
+}
+
+
 QStringList RbSettings::platforms()
 {
     ensureRbSettingsExists();
@@ -250,6 +269,7 @@ QStringList RbSettings::languages()
     return result;
 }
 
+
 QString RbSettings::name(QString platform)
 {
     ensureRbSettingsExists();