The-Homebrew-Cloud/dolphin
2
0
Fork 0
mirror of https://github.com/dolphin-emu/dolphin.git synced 2025-03-12 06:39:14 +01:00
Code Issues Packages Projects Releases Wiki Activity

InputCommon/XInput2: Changed axis value reading to be more correct

Browse Source
This commit is contained in:
TheConfuZzledDude 2022-12-26 17:27:48 +00:00
parent 27d596b89c
commit 288fa635f9
2 changed files with 39 additions and 30 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

66
Source/Core/InputCommon/ControllerInterface/Xlib/XInput2.cpp
View File

@ -92,8 +92,8 @@ void PopulateDevices(void* const hwnd)
if (current_master->use == XIMasterPointer) if (current_master->use == XIMasterPointer)
{ {
// We need to query the master for the scroll wheel's increment, since the increment used varies // We need to query the master for the scroll wheel's increment, since the increment used
// depending on what input driver is being used. For example, xf86-libinput uses 120.0. // varies depending on what input driver is being used. For example, xf86-libinput uses 120.0.
for (int j = 0; j < current_master->num_classes; j++) for (int j = 0; j < current_master->num_classes; j++)
{ {
if (current_master->classes[j]->type == XIScrollClass) if (current_master->classes[j]->type == XIScrollClass)
@ -106,8 +106,9 @@ void PopulateDevices(void* const hwnd)
} }
// Since current_master is a master pointer, its attachment must // Since current_master is a master pointer, its attachment must
// be a master keyboard. // be a master keyboard.
g_controller_interface.AddDevice(std::make_shared<KeyboardMouse>( g_controller_interface.AddDevice(
(Window)hwnd, xi_opcode, current_master->deviceid, current_master->attachment, scroll_increment)); std::make_shared<KeyboardMouse>((Window)hwnd, xi_opcode, current_master->deviceid,
current_master->attachment, scroll_increment));
} }
} }
@ -149,8 +150,10 @@ void KeyboardMouse::SelectEventsForDevice(XIEventMask* mask, int deviceid)
XIFreeDeviceInfo(all_slaves); XIFreeDeviceInfo(all_slaves);
} }
KeyboardMouse::KeyboardMouse(Window window, int opcode, int pointer, int keyboard, double scroll_increment) KeyboardMouse::KeyboardMouse(Window window, int opcode, int pointer, int keyboard,
: m_window(window), xi_opcode(opcode), pointer_deviceid(pointer), keyboard_deviceid(keyboard), scroll_increment(scroll_increment) double scroll_increment)
: m_window(window), xi_opcode(opcode), pointer_deviceid(pointer), keyboard_deviceid(keyboard),
scroll_increment(scroll_increment)
{ {
// The cool thing about each KeyboardMouse object having its own Display // The cool thing about each KeyboardMouse object having its own Display
// is that each one gets its own separate copy of the X11 event stream, // is that each one gets its own separate copy of the X11 event stream,
@ -321,34 +324,39 @@ void KeyboardMouse::UpdateInput()
m_state.keyboard[dev_event->detail / 8] &= ~(1 << (dev_event->detail % 8)); m_state.keyboard[dev_event->detail / 8] &= ~(1 << (dev_event->detail % 8));
break; break;
case XI_RawMotion: case XI_RawMotion:
{
mouse_moved = true; mouse_moved = true;
// always safe because there is always at least one byte in float values[4] = {};
// raw_event->valuators.mask, and if a bit is set in the mask, size_t value_idx = 0;
// then the value in raw_values is also available.
if (XIMaskIsSet(raw_event->valuators.mask, 0)) // We only care about the first 4 axes, which should always be available at minimum
for (int i = 0; i < 4; ++i)
{ {
delta_delta = raw_event->raw_values[0]; if (XIMaskIsSet(raw_event->valuators.mask, i))
// test for inf and nan {
if (delta_delta == delta_delta && 1 + delta_delta != delta_delta) values[i] = raw_event->raw_values[value_idx++];
delta_x += delta_delta; }
}
if (XIMaskIsSet(raw_event->valuators.mask, 1))
{
delta_delta = raw_event->raw_values[1];
// test for inf and nan
if (delta_delta == delta_delta && 1 + delta_delta != delta_delta)
delta_y += delta_delta;
}
if (XIMaskIsSet(raw_event->valuators.mask, 3))
{
// Scroll wheel input gets scaled to be similar to the mouse axes
delta_delta = raw_event->raw_values[0] * 8.0 / scroll_increment;
// test for inf and nan
if (delta_delta == delta_delta && 1 + delta_delta != delta_delta)
delta_z += delta_delta;
} }
delta_delta = values[0];
// test for inf and nan
if (delta_delta == delta_delta && 1 + delta_delta != delta_delta)
delta_x += delta_delta;
delta_delta = values[1];
// test for inf and nan
if (delta_delta == delta_delta && 1 + delta_delta != delta_delta)
delta_y += delta_delta;
// Scroll wheel input gets scaled to be similar to the mouse axes
delta_delta = values[3] * 8.0 / scroll_increment;
// test for inf and nan
if (delta_delta == delta_delta && 1 + delta_delta != delta_delta)
delta_z += delta_delta;
break; break;
}
case XI_FocusOut: case XI_FocusOut:
// Clear keyboard state on FocusOut as we will not be receiving KeyRelease events. // Clear keyboard state on FocusOut as we will not be receiving KeyRelease events.
m_state.keyboard.fill(0); m_state.keyboard.fill(0);

3
Source/Core/InputCommon/ControllerInterface/Xlib/XInput2.h
View File

@ -113,7 +113,8 @@ private:
public: public:
void UpdateInput() override; void UpdateInput() override;
KeyboardMouse(Window window, int opcode, int pointer_deviceid, int keyboard_deviceid, double scroll_increment); KeyboardMouse(Window window, int opcode, int pointer_deviceid, int keyboard_deviceid,
double scroll_increment);
~KeyboardMouse(); ~KeyboardMouse();
std::string GetName() const override; std::string GetName() const override;