actions/itgmania212121
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Changed DirectInput mouse input to use an if statement instead of a while loop to avoid input lockup.

Browse Source
This commit is contained in:
Kyzentun Keeslala
2015-10-09 23:54:35 -06:00
parent de1313ab4d
commit 7a26044934
1 changed files with 10 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/arch/InputHandler/InputHandler_DirectInput.cpp
+10 -2
View File
@@ -539,7 +539,14 @@ void InputHandler_DInput::UpdateBuffered( DIDevice &device, const RageTimer &tm
{ {
DeviceInput diUp = DeviceInput(dev, up, 1.0f, tm); DeviceInput diUp = DeviceInput(dev, up, 1.0f, tm);
DeviceInput diDown = DeviceInput(dev, down, 0.0f, tm); DeviceInput diDown = DeviceInput(dev, down, 0.0f, tm);
while( fWheelDelta >= WHEEL_DELTA ) // This if statement used to be a while loop. But Kevin
// reported that scrolling the mouse wheel locked up input.
// I assume that fWheelDelta was some absurdly large value,
// causing an infinite loop. Changing the while loop to an
// if fixed the input lockup. Anything that wants to
// imitate multiple presses when the wheel is scrolled will
// have to do it manually. -Kyz
if( fWheelDelta >= WHEEL_DELTA )
{ {
ButtonPressed( diUp ); ButtonPressed( diUp );
ButtonPressed( diDown ); ButtonPressed( diDown );
@@ -551,7 +558,8 @@ void InputHandler_DInput::UpdateBuffered( DIDevice &device, const RageTimer &tm
{ {
DeviceInput diDown = DeviceInput(dev, down, 1.0f, tm); DeviceInput diDown = DeviceInput(dev, down, 1.0f, tm);
DeviceInput diUp = DeviceInput(dev, up, 0.0f, tm); DeviceInput diUp = DeviceInput(dev, up, 0.0f, tm);
while( fWheelDelta <= -WHEEL_DELTA ) // See comment for the l > 0 case. -Kyz
if( fWheelDelta <= -WHEEL_DELTA )
{ {
ButtonPressed( diDown ); ButtonPressed( diDown );
ButtonPressed( diUp ); ButtonPressed( diUp );