api/joy: refresh only manually on certain events

Refresh joylist when new listener arrives, and when the joy singleton
just gets created. Enumerating joys all the time causes high CPU usage
for some of the users.

Issue: #279
Backtrace-by: @exulant
Reported-by: @aka2k

1 files changed, 2 insertions, 6 deletions

diff --git a/opentrack/win32-joystick.hpp b/opentrack/win32-joystick.hpp
index 6d5e1074..9c888326 100644
--- a/opentrack/win32-joystick.hpp
+++ b/opentrack/win32-joystick.hpp
@@ -57,18 +57,14 @@ struct OPENTRACK_EXPORT win32_joy_ctx
     
     win32_joy_ctx();
     ~win32_joy_ctx();
-    
+    void refresh();
 private:
-    enum { joylist_refresh_ms = 100 };
-    
     QMutex mtx;
-    Timer timer_joylist;
     QMainWindow fake_main_window;
     LPDIRECTINPUT8 di;
     
     static QString guid_to_string(const GUID guid);
     void release();
-    void refresh(bool first);
     
     struct joy
     {
@@ -97,7 +93,7 @@ private:
         static BOOL CALLBACK EnumJoysticksCallback(const DIDEVICEINSTANCE* pdidInstance, VOID* pContext);
         static BOOL CALLBACK EnumObjectsCallback(const DIDEVICEOBJECTINSTANCE* pdidoi, VOID* ctx);
     public:
-        enum_state(std::unordered_map<QString, std::shared_ptr<joy>>& joys, QMutex &mtx, QMainWindow& fake_main_window, LPDIRECTINPUT8 di);
+        enum_state(std::unordered_map<QString, std::shared_ptr<joy>>& joys, QMainWindow& fake_main_window, LPDIRECTINPUT8 di);
     };
 };