Просмотр исходного кода

Merge pull request #1440 from lioncash/array

ui_settings: Place definition of the theme array within the cpp file
bunnei 7 лет назад
Родитель
Сommit
e6ee31a8e9
3 измененных файлов с 10 добавлено и 5 удалено
  1. 1 1
      src/yuzu/configuration/configure_general.cpp
  2. 7 1
      src/yuzu/ui_settings.cpp
  3. 2 3
      src/yuzu/ui_settings.h

+ 1 - 1
src/yuzu/configuration/configure_general.cpp

@@ -13,7 +13,7 @@ ConfigureGeneral::ConfigureGeneral(QWidget* parent)
 
 
     ui->setupUi(this);
     ui->setupUi(this);
 
 
-    for (auto theme : UISettings::themes) {
+    for (const auto& theme : UISettings::themes) {
         ui->theme_combobox->addItem(theme.first, theme.second);
         ui->theme_combobox->addItem(theme.first, theme.second);
     }
     }
 
 

+ 7 - 1
src/yuzu/ui_settings.cpp

@@ -6,5 +6,11 @@
 
 
 namespace UISettings {
 namespace UISettings {
 
 
+const Themes themes{{
+    {"Default", "default"},
+    {"Dark", "qdarkstyle"},
+}};
+
 Values values = {};
 Values values = {};
-}
+
+} // namespace UISettings

+ 2 - 3
src/yuzu/ui_settings.h

@@ -15,9 +15,8 @@ namespace UISettings {
 using ContextualShortcut = std::pair<QString, int>;
 using ContextualShortcut = std::pair<QString, int>;
 using Shortcut = std::pair<QString, ContextualShortcut>;
 using Shortcut = std::pair<QString, ContextualShortcut>;
 
 
-static const std::array<std::pair<QString, QString>, 2> themes = {
-    {std::make_pair(QString("Default"), QString("default")),
-     std::make_pair(QString("Dark"), QString("qdarkstyle"))}};
+using Themes = std::array<std::pair<const char*, const char*>, 2>;
+extern const Themes themes;
 
 
 struct Values {
 struct Values {
     QByteArray geometry;
     QByteArray geometry;