diff options
| author | 2023-05-09 01:29:08 -0400 | |
|---|---|---|
| committer | 2023-07-21 10:56:07 -0400 | |
| commit | ea4afbfc54319b6fb4d0f4546da048bf68bdad3b (patch) | |
| tree | 5ebacec406e3c0ad591ae4b0cd933a979f276081 /src | |
| parent | configure_graphics: Remove redundant log (diff) | |
| download | yuzu-ea4afbfc54319b6fb4d0f4546da048bf68bdad3b.tar.gz yuzu-ea4afbfc54319b6fb4d0f4546da048bf68bdad3b.tar.xz yuzu-ea4afbfc54319b6fb4d0f4546da048bf68bdad3b.zip | |
config: Don't merge the maps
Me shooting myself in the foot 3 days in advance.
Diffstat (limited to 'src')
| -rw-r--r-- | src/yuzu/configuration/config.cpp | 19 | ||||
| -rw-r--r-- | src/yuzu/configuration/config.h | 2 |
2 files changed, 10 insertions, 11 deletions
diff --git a/src/yuzu/configuration/config.cpp b/src/yuzu/configuration/config.cpp index 08aa20859..28c606f5e 100644 --- a/src/yuzu/configuration/config.cpp +++ b/src/yuzu/configuration/config.cpp | |||
| @@ -19,10 +19,6 @@ namespace FS = Common::FS; | |||
| 19 | 19 | ||
| 20 | Config::Config(const std::string& config_name, ConfigType config_type) | 20 | Config::Config(const std::string& config_name, ConfigType config_type) |
| 21 | : type(config_type), global{config_type == ConfigType::GlobalConfig} { | 21 | : type(config_type), global{config_type == ConfigType::GlobalConfig} { |
| 22 | |||
| 23 | settings_map = Settings::values.linkage.by_category; | ||
| 24 | settings_map.merge(UISettings::values.linkage.by_category); | ||
| 25 | |||
| 26 | Initialize(config_name); | 22 | Initialize(config_name); |
| 27 | } | 23 | } |
| 28 | 24 | ||
| @@ -1311,14 +1307,22 @@ const std::string& Config::GetConfigFilePath() const { | |||
| 1311 | return qt_config_loc; | 1307 | return qt_config_loc; |
| 1312 | } | 1308 | } |
| 1313 | 1309 | ||
| 1310 | static auto FindRelevantList(Settings::Category category) { | ||
| 1311 | auto& map = Settings::values.linkage.by_category; | ||
| 1312 | if (map.contains(category)) { | ||
| 1313 | return Settings::values.linkage.by_category[category]; | ||
| 1314 | } | ||
| 1315 | return UISettings::values.linkage.by_category[category]; | ||
| 1316 | } | ||
| 1317 | |||
| 1314 | void Config::ReadCategory(Settings::Category category) { | 1318 | void Config::ReadCategory(Settings::Category category) { |
| 1315 | const auto& settings = settings_map[category]; | 1319 | const auto& settings = FindRelevantList(category); |
| 1316 | std::for_each(settings.begin(), settings.end(), | 1320 | std::for_each(settings.begin(), settings.end(), |
| 1317 | [&](const auto& setting) { ReadSettingGeneric(setting); }); | 1321 | [&](const auto& setting) { ReadSettingGeneric(setting); }); |
| 1318 | } | 1322 | } |
| 1319 | 1323 | ||
| 1320 | void Config::WriteCategory(Settings::Category category) { | 1324 | void Config::WriteCategory(Settings::Category category) { |
| 1321 | const auto& settings = settings_map[category]; | 1325 | const auto& settings = FindRelevantList(category); |
| 1322 | std::for_each(settings.begin(), settings.end(), | 1326 | std::for_each(settings.begin(), settings.end(), |
| 1323 | [&](const auto& setting) { WriteSettingGeneric(setting); }); | 1327 | [&](const auto& setting) { WriteSettingGeneric(setting); }); |
| 1324 | } | 1328 | } |
| @@ -1346,11 +1350,8 @@ void Config::ReadSettingGeneric(Settings::BasicSetting* const setting) { | |||
| 1346 | 1350 | ||
| 1347 | void Config::WriteSettingGeneric(Settings::BasicSetting* const setting) const { | 1351 | void Config::WriteSettingGeneric(Settings::BasicSetting* const setting) const { |
| 1348 | if (!setting->Save()) { | 1352 | if (!setting->Save()) { |
| 1349 | LOG_DEBUG(Frontend, "Skipping \"{}\" marked for not saving", setting->GetLabel()); | ||
| 1350 | return; | 1353 | return; |
| 1351 | } | 1354 | } |
| 1352 | LOG_DEBUG(Frontend, "Saving {} setting \"{}\"...", global ? "global" : "custom", | ||
| 1353 | setting->GetLabel()); | ||
| 1354 | const QVariant value = QVariant::fromValue(QString::fromStdString(setting->ToString())); | 1355 | const QVariant value = QVariant::fromValue(QString::fromStdString(setting->ToString())); |
| 1355 | const QVariant default_value = | 1356 | const QVariant default_value = |
| 1356 | QVariant::fromValue(QString::fromStdString(setting->DefaultToString())); | 1357 | QVariant::fromValue(QString::fromStdString(setting->DefaultToString())); |
diff --git a/src/yuzu/configuration/config.h b/src/yuzu/configuration/config.h index ec748bf0c..553a82295 100644 --- a/src/yuzu/configuration/config.h +++ b/src/yuzu/configuration/config.h | |||
| @@ -204,8 +204,6 @@ private: | |||
| 204 | void ReadSettingGeneric(Settings::BasicSetting* const setting); | 204 | void ReadSettingGeneric(Settings::BasicSetting* const setting); |
| 205 | void WriteSettingGeneric(Settings::BasicSetting* const setting) const; | 205 | void WriteSettingGeneric(Settings::BasicSetting* const setting) const; |
| 206 | 206 | ||
| 207 | std::map<Settings::Category, std::forward_list<Settings::BasicSetting*>> settings_map; | ||
| 208 | |||
| 209 | const ConfigType type; | 207 | const ConfigType type; |
| 210 | std::unique_ptr<QSettings> qt_config; | 208 | std::unique_ptr<QSettings> qt_config; |
| 211 | std::string qt_config_loc; | 209 | std::string qt_config_loc; |