From 17849b12f232c89bdae16f7e179d3a92a32280ad Mon Sep 17 00:00:00 2001
From: Stanislaw Halik <sthalik@misaki.pl>
Date: Wed, 11 Nov 2015 13:44:45 +0100
Subject: shortcuts: map joystick buttons on depress only

Some buttons like the X65 mode switch are held all the time. Prevent
them from hogging all the keybindings.

Issue: #118
---
 opentrack/win32-joystick-shortcuts.hpp | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

(limited to 'opentrack/win32-joystick-shortcuts.hpp')

diff --git a/opentrack/win32-joystick-shortcuts.hpp b/opentrack/win32-joystick-shortcuts.hpp
index 67465bce..3c839197 100644
--- a/opentrack/win32-joystick-shortcuts.hpp
+++ b/opentrack/win32-joystick-shortcuts.hpp
@@ -16,7 +16,7 @@
 
 struct win32_joy_ctx
 {
-    using fn = std::function<void(const QString& guid, int btn)>;
+    using fn = std::function<void(const QString& guid, int btn, bool held)>;
     
     void poll(fn f)
     {
@@ -32,10 +32,13 @@ struct win32_joy_ctx
     {
         LPDIRECTINPUTDEVICE8 joy_handle;
         QString guid;
+        bool pressed[128];
         
         joy(LPDIRECTINPUTDEVICE8 handle, const QString& guid) : joy_handle(handle), guid(guid)
         {
             qDebug() << "got joy" << guid;
+            for (int i = 0; i < 128; i++)
+                pressed[i] = false;
         }
         
         ~joy()
@@ -88,8 +91,12 @@ struct win32_joy_ctx
             }
             
             for (int i = 0; i < 128; i++)
-                if (js.rgbButtons[i] & 0x80)
-                    f(guid, i);
+            {
+                const bool state = !!(js.rgbButtons[i] & 0x80);
+                if (state != pressed[i])
+                    f(guid, i, state);
+                pressed[i] = state;
+            }
             
             return true;
         }
-- 
cgit v1.2.3