commit a2d3bf71c1a1650edf0480c845e24c862ded8cb8
author Ryan C. Gordon <icculus@icculus.org> Wed Jun 21 01:22:00 2017 -0400
committer Ryan C. Gordon <icculus@icculus.org> Wed Jun 21 01:22:00 2017 -0400
tree 87da9e4a2adffdee0fb49c43a7f43f158c398845
parent 3d08ff557311d8bc5775c369c6162847335473f0

xinput: use the full range of the haptic motors (thanks, Trent!).

XInput goes from 0 to 64k; we were feeding it values in the range of 0 to 32k.

Fixes Bugzilla #3002.

src/haptic/windows/SDL_xinputhaptic.c

diff --git a/src/haptic/windows/SDL_xinputhaptic.c b/src/haptic/windows/SDL_xinputhaptic.c
index 6832f7e..419a84c 100644
--- a/src/haptic/windows/SDL_xinputhaptic.c
+++ b/src/haptic/windows/SDL_xinputhaptic.c
@@ -278,8 +278,9 @@
 {
     XINPUT_VIBRATION *vib = &effect->hweffect->vibration;
     SDL_assert(data->type == SDL_HAPTIC_LEFTRIGHT);
-    vib->wLeftMotorSpeed = data->leftright.large_magnitude;
-    vib->wRightMotorSpeed = data->leftright.small_magnitude;
+    /* SDL_HapticEffect has max magnitude of 32767, XInput expects 65535 max, so multiply */
+    vib->wLeftMotorSpeed = data->leftright.large_magnitude * 2;
+    vib->wRightMotorSpeed = data->leftright.small_magnitude * 2;
     SDL_LockMutex(haptic->hwdata->mutex);
     if (haptic->hwdata->stopTicks) {  /* running right now? Update it. */
         XINPUTSETSTATE(haptic->hwdata->userid, vib);