From 50153a1cb2edbb9ae3aef79a464779099c1e2827 Mon Sep 17 00:00:00 2001 From: Zach Hilman Date: Wed, 13 Jun 2018 13:01:05 -0400 Subject: Avoid initializing single-joycon layouts with handheld controller --- src/core/hle/service/hid/hid.cpp | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) (limited to 'src/core/hle/service/hid/hid.cpp') diff --git a/src/core/hle/service/hid/hid.cpp b/src/core/hle/service/hid/hid.cpp index 85ca4bf06..d84c0a0b2 100644 --- a/src/core/hle/service/hid/hid.cpp +++ b/src/core/hle/service/hid/hid.cpp @@ -83,7 +83,10 @@ private: controller_header.left_color_buttons = JOYCON_BUTTONS_NEON_BLUE; for (size_t controller = 0; controller < mem.controllers.size(); controller++) { - for (int index = 0; index < HID_NUM_LAYOUTS; index++) { + for (int index = 0; + index < + (controller != Controller_Handheld ? HID_NUM_LAYOUTS : HID_NUM_LAYOUTS_HANDHELD); + index++) { ControllerLayout& layout = mem.controllers[controller].layouts[index]; layout.header.num_entries = HID_NUM_ENTRIES; layout.header.max_entry_index = HID_NUM_ENTRIES - 1; -- cgit v1.2.3 From a353322b58afa439eabce0694f457381fd80a7fe Mon Sep 17 00:00:00 2001 From: Zach Hilman Date: Wed, 13 Jun 2018 13:44:46 -0400 Subject: Move loop condition to free function --- src/core/hle/service/hid/hid.cpp | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) (limited to 'src/core/hle/service/hid/hid.cpp') diff --git a/src/core/hle/service/hid/hid.cpp b/src/core/hle/service/hid/hid.cpp index d84c0a0b2..d4f741505 100644 --- a/src/core/hle/service/hid/hid.cpp +++ b/src/core/hle/service/hid/hid.cpp @@ -16,6 +16,14 @@ namespace Service::HID { +int GetNumberLayoutsForController(size_t controller_id) { + if (controller_id == Controller_Handheld) { + return HID_NUM_LAYOUTS_HANDHELD; + } else { + return HID_NUM_LAYOUTS; + } +} + // Updating period for each HID device. // TODO(shinyquagsire23): These need better values. constexpr u64 pad_update_ticks = CoreTiming::BASE_CLOCK_RATE / 10000; @@ -83,10 +91,7 @@ private: controller_header.left_color_buttons = JOYCON_BUTTONS_NEON_BLUE; for (size_t controller = 0; controller < mem.controllers.size(); controller++) { - for (int index = 0; - index < - (controller != Controller_Handheld ? HID_NUM_LAYOUTS : HID_NUM_LAYOUTS_HANDHELD); - index++) { + for (int index = 0; index < GetNumberLayoutsForController(controller); index++) { ControllerLayout& layout = mem.controllers[controller].layouts[index]; layout.header.num_entries = HID_NUM_ENTRIES; layout.header.max_entry_index = HID_NUM_ENTRIES - 1; -- cgit v1.2.3 From ac88d3e89f68cf9eac11014225c2847e96f8a620 Mon Sep 17 00:00:00 2001 From: Zach Hilman Date: Wed, 13 Jun 2018 20:03:12 -0400 Subject: Narrow down filter of layout configs --- src/core/hle/service/hid/hid.cpp | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) (limited to 'src/core/hle/service/hid/hid.cpp') diff --git a/src/core/hle/service/hid/hid.cpp b/src/core/hle/service/hid/hid.cpp index d4f741505..2696a8bf0 100644 --- a/src/core/hle/service/hid/hid.cpp +++ b/src/core/hle/service/hid/hid.cpp @@ -16,14 +16,6 @@ namespace Service::HID { -int GetNumberLayoutsForController(size_t controller_id) { - if (controller_id == Controller_Handheld) { - return HID_NUM_LAYOUTS_HANDHELD; - } else { - return HID_NUM_LAYOUTS; - } -} - // Updating period for each HID device. // TODO(shinyquagsire23): These need better values. constexpr u64 pad_update_ticks = CoreTiming::BASE_CLOCK_RATE / 10000; @@ -91,7 +83,11 @@ private: controller_header.left_color_buttons = JOYCON_BUTTONS_NEON_BLUE; for (size_t controller = 0; controller < mem.controllers.size(); controller++) { - for (int index = 0; index < GetNumberLayoutsForController(controller); index++) { + for (int index = 0; index < HID_NUM_LAYOUTS; index++) { + // TODO(DarkLordZach): Is this layout/controller config actually invalid? + if (controller == Controller_Handheld && index == Layout_Single) + continue; + ControllerLayout& layout = mem.controllers[controller].layouts[index]; layout.header.num_entries = HID_NUM_ENTRIES; layout.header.max_entry_index = HID_NUM_ENTRIES - 1; -- cgit v1.2.3