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/keybinding-worker.cpp        |  5 +++--
 opentrack/keybinding-worker.hpp        |  3 ++-
 opentrack/win32-joystick-shortcuts.hpp | 13 ++++++++++---
 3 files changed, 15 insertions(+), 6 deletions(-)

(limited to 'opentrack')

diff --git a/opentrack/keybinding-worker.cpp b/opentrack/keybinding-worker.cpp
index e6c023ef..e9255801 100644
--- a/opentrack/keybinding-worker.cpp
+++ b/opentrack/keybinding-worker.cpp
@@ -73,12 +73,13 @@ void KeybindingWorker::run() {
     while (!should_quit)
     {
         {
-            using joy_fn = std::function<void(const QString& guid, int idx)>;
+            using joy_fn = std::function<void(const QString& guid, int idx, bool held)>;
             
-            joy_fn f = [&](const QString& guid, int idx) -> void {
+            joy_fn f = [&](const QString& guid, int idx, bool held) -> void {
                 Key k;
                 k.keycode = idx;
                 k.guid = guid;
+                k.held = held;
                 receiver(k);
             };
             
diff --git a/opentrack/keybinding-worker.hpp b/opentrack/keybinding-worker.hpp
index 8cf59d65..cb3e5f9f 100644
--- a/opentrack/keybinding-worker.hpp
+++ b/opentrack/keybinding-worker.hpp
@@ -32,9 +32,10 @@ struct Key {
     bool shift;
     bool ctrl;
     bool alt;
+    bool held;
     Timer timer;
 public:
-    Key() : keycode(0), shift(false), ctrl(false), alt(false)
+    Key() : keycode(0), shift(false), ctrl(false), alt(false), held(true)
     {
     }
 
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