From 387ede76d2e1e427f6722cbe19a018c95d762748 Mon Sep 17 00:00:00 2001 From: lat9nq <22451773+lat9nq@users.noreply.github.com> Date: Mon, 21 Aug 2023 16:03:30 -0400 Subject: general: Convert use_docked_mode to an enumeration Allows some special interactions with it in the Qt frontend. --- src/yuzu/main.cpp | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) (limited to 'src/yuzu/main.cpp') diff --git a/src/yuzu/main.cpp b/src/yuzu/main.cpp index a9d035f3d..8ab4e73b1 100644 --- a/src/yuzu/main.cpp +++ b/src/yuzu/main.cpp @@ -1158,9 +1158,9 @@ void GMainWindow::InitializeWidgets() { [this](const QPoint& menu_location) { QMenu context_menu; - for (auto const& docked_mode_pair : Config::use_docked_mode_texts_map) { - context_menu.addAction(docked_mode_pair.second, [this, docked_mode_pair] { - if (docked_mode_pair.first != Settings::values.use_docked_mode.GetValue()) { + for (auto const& [value, text] : Config::use_docked_mode_texts_map) { + context_menu.addAction(text, [this, value] { + if (value != Settings::values.use_docked_mode.GetValue()) { OnToggleDockedMode(); } }); @@ -3636,7 +3636,8 @@ void GMainWindow::OnTasReset() { } void GMainWindow::OnToggleDockedMode() { - const bool is_docked = Settings::values.use_docked_mode.GetValue(); + const bool is_docked = + Settings::values.use_docked_mode.GetValue() == Settings::ConsoleMode::Docked; auto* player_1 = system->HIDCore().GetEmulatedController(Core::HID::NpadIdType::Player1); auto* handheld = system->HIDCore().GetEmulatedController(Core::HID::NpadIdType::Handheld); @@ -3650,7 +3651,8 @@ void GMainWindow::OnToggleDockedMode() { controller_dialog->refreshConfiguration(); } - Settings::values.use_docked_mode.SetValue(!is_docked); + Settings::values.use_docked_mode.SetValue(is_docked ? Settings::ConsoleMode::Docked + : Settings::ConsoleMode::Handheld); UpdateDockedButton(); OnDockedModeChanged(is_docked, !is_docked, *system); } @@ -4080,8 +4082,8 @@ void GMainWindow::UpdateGPUAccuracyButton() { } void GMainWindow::UpdateDockedButton() { - const bool is_docked = Settings::values.use_docked_mode.GetValue(); - dock_status_button->setChecked(is_docked); + const auto is_docked = Settings::values.use_docked_mode.GetValue(); + dock_status_button->setChecked(is_docked == Settings::ConsoleMode::Docked); dock_status_button->setText( Config::use_docked_mode_texts_map.find(is_docked)->second.toUpper()); } -- cgit v1.2.3 From 3c45452fae7f33d0534c144c5f588ef98f0a1346 Mon Sep 17 00:00:00 2001 From: lat9nq <22451773+lat9nq@users.noreply.github.com> Date: Tue, 22 Aug 2023 21:58:23 -0400 Subject: general: Use console mode helper across project --- src/yuzu/main.cpp | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) (limited to 'src/yuzu/main.cpp') diff --git a/src/yuzu/main.cpp b/src/yuzu/main.cpp index 8ab4e73b1..8fe8d0a23 100644 --- a/src/yuzu/main.cpp +++ b/src/yuzu/main.cpp @@ -3636,8 +3636,7 @@ void GMainWindow::OnTasReset() { } void GMainWindow::OnToggleDockedMode() { - const bool is_docked = - Settings::values.use_docked_mode.GetValue() == Settings::ConsoleMode::Docked; + const bool is_docked = Settings::IsDockedMode(); auto* player_1 = system->HIDCore().GetEmulatedController(Core::HID::NpadIdType::Player1); auto* handheld = system->HIDCore().GetEmulatedController(Core::HID::NpadIdType::Handheld); @@ -4082,10 +4081,10 @@ void GMainWindow::UpdateGPUAccuracyButton() { } void GMainWindow::UpdateDockedButton() { - const auto is_docked = Settings::values.use_docked_mode.GetValue(); - dock_status_button->setChecked(is_docked == Settings::ConsoleMode::Docked); + const auto console_mode = Settings::values.use_docked_mode.GetValue(); + dock_status_button->setChecked(console_mode == Settings::ConsoleMode::Docked); dock_status_button->setText( - Config::use_docked_mode_texts_map.find(is_docked)->second.toUpper()); + Config::use_docked_mode_texts_map.find(console_mode)->second.toUpper()); } void GMainWindow::UpdateAPIText() { -- cgit v1.2.3 From ce0f1baf5136dc0ec136318b216976fb34bacb30 Mon Sep 17 00:00:00 2001 From: lat9nq <22451773+lat9nq@users.noreply.github.com> Date: Tue, 22 Aug 2023 22:35:55 -0400 Subject: main: Access by reference Old Clang is fussy about this. --- src/yuzu/main.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/yuzu/main.cpp') diff --git a/src/yuzu/main.cpp b/src/yuzu/main.cpp index 8fe8d0a23..3f126d9d1 100644 --- a/src/yuzu/main.cpp +++ b/src/yuzu/main.cpp @@ -1159,7 +1159,7 @@ void GMainWindow::InitializeWidgets() { QMenu context_menu; for (auto const& [value, text] : Config::use_docked_mode_texts_map) { - context_menu.addAction(text, [this, value] { + context_menu.addAction(text, [&] { if (value != Settings::values.use_docked_mode.GetValue()) { OnToggleDockedMode(); } -- cgit v1.2.3 From 1cdd11d9f5541ba9b49baf3dddcb7e411f035669 Mon Sep 17 00:00:00 2001 From: lat9nq <22451773+lat9nq@users.noreply.github.com> Date: Wed, 23 Aug 2023 14:26:34 -0400 Subject: main: Fix docked mode button, clang 14 error --- src/yuzu/main.cpp | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) (limited to 'src/yuzu/main.cpp') diff --git a/src/yuzu/main.cpp b/src/yuzu/main.cpp index 3f126d9d1..99961ca3e 100644 --- a/src/yuzu/main.cpp +++ b/src/yuzu/main.cpp @@ -1158,9 +1158,9 @@ void GMainWindow::InitializeWidgets() { [this](const QPoint& menu_location) { QMenu context_menu; - for (auto const& [value, text] : Config::use_docked_mode_texts_map) { - context_menu.addAction(text, [&] { - if (value != Settings::values.use_docked_mode.GetValue()) { + for (auto const& pair : Config::use_docked_mode_texts_map) { + context_menu.addAction(pair.second, [this, &pair] { + if (pair.first != Settings::values.use_docked_mode.GetValue()) { OnToggleDockedMode(); } }); @@ -3650,8 +3650,8 @@ void GMainWindow::OnToggleDockedMode() { controller_dialog->refreshConfiguration(); } - Settings::values.use_docked_mode.SetValue(is_docked ? Settings::ConsoleMode::Docked - : Settings::ConsoleMode::Handheld); + Settings::values.use_docked_mode.SetValue(is_docked ? Settings::ConsoleMode::Handheld + : Settings::ConsoleMode::Docked); UpdateDockedButton(); OnDockedModeChanged(is_docked, !is_docked, *system); } @@ -4082,7 +4082,7 @@ void GMainWindow::UpdateGPUAccuracyButton() { void GMainWindow::UpdateDockedButton() { const auto console_mode = Settings::values.use_docked_mode.GetValue(); - dock_status_button->setChecked(console_mode == Settings::ConsoleMode::Docked); + dock_status_button->setChecked(Settings::IsDockedMode()); dock_status_button->setText( Config::use_docked_mode_texts_map.find(console_mode)->second.toUpper()); } -- cgit v1.2.3