summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMorph <39850852+Morph1984@users.noreply.github.com>2020-08-28 19:14:19 +0200
committerMorph <39850852+Morph1984@users.noreply.github.com>2020-09-04 18:23:26 +0200
commitb65456b958f64dee6215962b11430f57f89dd5a8 (patch)
tree141c77c3844a1d82aa89c4d6802a6307df18e272
parentAddress feedback (diff)
downloadyuzu-b65456b958f64dee6215962b11430f57f89dd5a8.tar
yuzu-b65456b958f64dee6215962b11430f57f89dd5a8.tar.gz
yuzu-b65456b958f64dee6215962b11430f57f89dd5a8.tar.bz2
yuzu-b65456b958f64dee6215962b11430f57f89dd5a8.tar.lz
yuzu-b65456b958f64dee6215962b11430f57f89dd5a8.tar.xz
yuzu-b65456b958f64dee6215962b11430f57f89dd5a8.tar.zst
yuzu-b65456b958f64dee6215962b11430f57f89dd5a8.zip
-rw-r--r--src/core/frontend/applets/controller.cpp3
-rw-r--r--src/yuzu/applets/controller.cpp12
2 files changed, 8 insertions, 7 deletions
diff --git a/src/core/frontend/applets/controller.cpp b/src/core/frontend/applets/controller.cpp
index 31a5cb2cc..4505da758 100644
--- a/src/core/frontend/applets/controller.cpp
+++ b/src/core/frontend/applets/controller.cpp
@@ -29,7 +29,8 @@ void DefaultControllerApplet::ReconfigureControllers(std::function<void()> callb
auto& players = Settings::values.players;
- const auto min_supported_players = parameters.enable_single_mode ? 1 : parameters.min_players;
+ const std::size_t min_supported_players =
+ parameters.enable_single_mode ? 1 : parameters.min_players;
// Disconnect Handheld first.
npad.DisconnectNPadAtIndex(8);
diff --git a/src/yuzu/applets/controller.cpp b/src/yuzu/applets/controller.cpp
index c960eb3dd..9d45f2a01 100644
--- a/src/yuzu/applets/controller.cpp
+++ b/src/yuzu/applets/controller.cpp
@@ -289,11 +289,9 @@ void QtControllerSelectorDialog::CallConfigureInputDialog() {
void QtControllerSelectorDialog::CheckIfParametersMet() {
// Here, we check and validate the current configuration against all applicable parameters.
- const auto& players = Settings::values.players;
-
- const auto num_connected_players =
+ const auto num_connected_players = static_cast<int>(
std::count_if(player_groupboxes.begin(), player_groupboxes.end(),
- [this](const QGroupBox* player) { return player->isChecked(); });
+ [this](const QGroupBox* player) { return player->isChecked(); }));
const auto min_supported_players = parameters.enable_single_mode ? 1 : parameters.min_players;
const auto max_supported_players = parameters.enable_single_mode ? 1 : parameters.max_players;
@@ -489,7 +487,8 @@ void QtControllerSelectorDialog::UpdateLEDPattern(std::size_t player_index) {
}
void QtControllerSelectorDialog::UpdateBorderColor(std::size_t player_index) {
- if (!parameters.enable_border_color || player_index >= parameters.max_players ||
+ if (!parameters.enable_border_color ||
+ player_index >= static_cast<std::size_t>(parameters.max_players) ||
player_groupboxes[player_index]->styleSheet().contains(QStringLiteral("QGroupBox"))) {
return;
}
@@ -506,7 +505,8 @@ void QtControllerSelectorDialog::UpdateBorderColor(std::size_t player_index) {
}
void QtControllerSelectorDialog::SetExplainText(std::size_t player_index) {
- if (!parameters.enable_explain_text || player_index >= parameters.max_players) {
+ if (!parameters.enable_explain_text ||
+ player_index >= static_cast<std::size_t>(parameters.max_players)) {
return;
}