From 7b7828a46ee4e313a3fb57ad51cd19f3d2909219 Mon Sep 17 00:00:00 2001 From: Sam Lantinga Date: Sat, 21 Jun 2014 21:30:49 -0700 Subject: [PATCH] You shouldn't get axis and hat events when your application doesn't have focus (unless you use the SDL_HINT_JOYSTICK_ALLOW_BACKGROUND_EVENTS hint) --- src/joystick/SDL_joystick.c | 38 ++++++++++++++++++++++--------------- 1 file changed, 23 insertions(+), 15 deletions(-) diff --git a/src/joystick/SDL_joystick.c b/src/joystick/SDL_joystick.c index b9924f3b6..ac71587a8 100644 --- a/src/joystick/SDL_joystick.c +++ b/src/joystick/SDL_joystick.c @@ -502,21 +502,22 @@ SDL_PrivateJoystickAxis(SDL_Joystick * joystick, Uint8 axis, Sint16 value) return 0; } + /* We ignore events if we don't have keyboard focus, except for centering + * events. + */ + if (SDL_PrivateJoystickShouldIgnoreEvent()) { + if ((value > 0 && value >= joystick->axes[axis]) || + (value < 0 && value <= joystick->axes[axis])) { + return 0; + } + } + /* Update internal joystick state */ if (value == joystick->axes[axis]) { return 0; } joystick->axes[axis] = value; - /* We ignore events if we don't have keyboard focus, except for centering - * events. - */ - if (SDL_PrivateJoystickShouldIgnoreEvent()) { - if (!(joystick->closed && joystick->uncentered)) { - return 0; - } - } - /* Post the event, if desired */ posted = 0; #if !SDL_EVENTS_DISABLED @@ -542,18 +543,20 @@ SDL_PrivateJoystickHat(SDL_Joystick * joystick, Uint8 hat, Uint8 value) return 0; } - /* Update internal joystick state */ - joystick->hats[hat] = value; - /* We ignore events if we don't have keyboard focus, except for centering * events. */ if (SDL_PrivateJoystickShouldIgnoreEvent()) { - if (!(joystick->closed && joystick->uncentered)) { + if (value != SDL_HAT_CENTERED) { return 0; } } + /* Update internal joystick state */ + if (value == joystick->hats[hat]) { + return 0; + } + joystick->hats[hat] = value; /* Post the event, if desired */ posted = 0; @@ -633,11 +636,16 @@ SDL_PrivateJoystickButton(SDL_Joystick * joystick, Uint8 button, Uint8 state) /* We ignore events if we don't have keyboard focus, except for button * release. */ - if (state == SDL_PRESSED && SDL_PrivateJoystickShouldIgnoreEvent()) { - return 0; + if (SDL_PrivateJoystickShouldIgnoreEvent()) { + if (state == SDL_PRESSED) { + return 0; + } } /* Update internal joystick state */ + if (state == joystick->buttons[button]) { + return 0; + } joystick->buttons[button] = state; /* Post the event, if desired */