diff --git a/src/citra_qt/configuration/config.cpp b/src/citra_qt/configuration/config.cpp index 9fe88e8929..95df2208b1 100644 --- a/src/citra_qt/configuration/config.cpp +++ b/src/citra_qt/configuration/config.cpp @@ -635,11 +635,11 @@ void Config::SaveValues() { qt_config->endGroup(); } -QVariant Config::ReadSetting(const QString& name) { +QVariant Config::ReadSetting(const QString& name) const { return qt_config->value(name); } -QVariant Config::ReadSetting(const QString& name, const QVariant& default_value) { +QVariant Config::ReadSetting(const QString& name, const QVariant& default_value) const { QVariant result; if (qt_config->value(name + "/default", false).toBool()) { result = default_value; diff --git a/src/citra_qt/configuration/config.h b/src/citra_qt/configuration/config.h index 70aab4f8c2..781926f3f7 100644 --- a/src/citra_qt/configuration/config.h +++ b/src/citra_qt/configuration/config.h @@ -27,8 +27,8 @@ public: private: void ReadValues(); void SaveValues(); - QVariant ReadSetting(const QString& name); - QVariant ReadSetting(const QString& name, const QVariant& default_value); + QVariant ReadSetting(const QString& name) const; + QVariant ReadSetting(const QString& name, const QVariant& default_value) const; void WriteSetting(const QString& name, const QVariant& value); void WriteSetting(const QString& name, const QVariant& value, const QVariant& default_value); diff --git a/src/citra_qt/configuration/configure_dialog.cpp b/src/citra_qt/configuration/configure_dialog.cpp index a0ef52d2e6..e1fc5abac3 100644 --- a/src/citra_qt/configuration/configure_dialog.cpp +++ b/src/citra_qt/configuration/configure_dialog.cpp @@ -2,7 +2,7 @@ // Licensed under GPLv2 or any later version // Refer to the license.txt file included. -#include +#include #include #include "citra_qt/configuration/config.h" #include "citra_qt/configuration/configure_dialog.h" @@ -74,7 +74,7 @@ void ConfigureDialog::PopulateSelectionList() { {tr("Controls"), {QT_TR_NOOP("Input"), QT_TR_NOOP("Hotkeys")}}}}; for (const auto& entry : items) { - auto* item = new QListWidgetItem(entry.first); + auto* const item = new QListWidgetItem(entry.first); item->setData(Qt::UserRole, entry.second); ui->selectorList->addItem(item); @@ -111,11 +111,11 @@ void ConfigureDialog::retranslateUi() { } void ConfigureDialog::UpdateVisibleTabs() { - auto items = ui->selectorList->selectedItems(); + const auto items = ui->selectorList->selectedItems(); if (items.isEmpty()) return; - const QHash widgets = { + const std::map widgets = { {"General", ui->generalTab}, {"System", ui->systemTab}, {"Input", ui->inputTab}, {"Hotkeys", ui->hotkeysTab}, {"Graphics", ui->graphicsTab}, {"Audio", ui->audioTab}, @@ -124,8 +124,8 @@ void ConfigureDialog::UpdateVisibleTabs() { ui->tabWidget->clear(); - QStringList tabs = items[0]->data(Qt::UserRole).toStringList(); + const QStringList tabs = items[0]->data(Qt::UserRole).toStringList(); for (const auto& tab : tabs) - ui->tabWidget->addTab(widgets[tab], tr(qPrintable(tab))); + ui->tabWidget->addTab(widgets.at(tab), tr(qPrintable(tab))); } diff --git a/src/citra_qt/configuration/configure_dialog.h b/src/citra_qt/configuration/configure_dialog.h index c902bf7020..91234ca0b5 100644 --- a/src/citra_qt/configuration/configure_dialog.h +++ b/src/citra_qt/configuration/configure_dialog.h @@ -21,8 +21,6 @@ public: ~ConfigureDialog() override; void applyConfiguration(); - void UpdateVisibleTabs(); - void PopulateSelectionList(); private slots: void onLanguageChanged(const QString& locale); @@ -33,6 +31,8 @@ signals: private: void setConfiguration(); void retranslateUi(); + void UpdateVisibleTabs(); + void PopulateSelectionList(); std::unique_ptr ui; HotkeyRegistry& registry; diff --git a/src/video_core/renderer_base.h b/src/video_core/renderer_base.h index ff9a71aba2..7cfcf0cef7 100644 --- a/src/video_core/renderer_base.h +++ b/src/video_core/renderer_base.h @@ -46,7 +46,11 @@ public: return rasterizer.get(); } - EmuWindow& GetRenderWindow() const { + EmuWindow& GetRenderWindow() { + return render_window; + } + + const EmuWindow& GetRenderWindow() const { return render_window; } diff --git a/src/video_core/video_core.cpp b/src/video_core/video_core.cpp index fa68e1d1fe..5bba1e3ac5 100644 --- a/src/video_core/video_core.cpp +++ b/src/video_core/video_core.cpp @@ -71,9 +71,9 @@ void RequestScreenshot(void* data, std::function callback, u16 GetResolutionScaleFactor() { if (g_hw_renderer_enabled) { - return !Settings::values.resolution_factor - ? g_renderer->GetRenderWindow().GetFramebufferLayout().GetScalingRatio() - : Settings::values.resolution_factor; + return Settings::values.resolution_factor + ? Settings::values.resolution_factor + : g_renderer->GetRenderWindow().GetFramebufferLayout().GetScalingRatio(); } else { // Software renderer always render at native resolution return 1;