Merge 662454b552 into 506c11ba43

* Wayland: Ignore key repeat events when no window has keyboard focus

CONTRIBUTORS.md

@@ -229,7 +229,7 @@ video tutorials.
  - Brandon Schaefer
  - Sebastian Schuberth
  - Scr3amer
- - Jan Schuerkamp
+ - Jan Schürkamp
  - Christian Sdunek
  - Matt Sealey
  - Steve Sexton

README.md

@@ -129,6 +129,7 @@ information on what to include when reporting a bug.
  - [Wayland] Bugfix: The fractional scaling related objects were not destroyed
  - [Wayland] Bugfix: `glfwInit` would segfault on compositor with no seat (#2517)
  - [Wayland] Bugfix: A drag entering a non-GLFW surface could cause a segfault
+ - [Wayland] Bugfix: Ignore key repeat events when no window has keyboard focus (#2727)
  - [X11] Bugfix: Running without a WM could trigger an assert (#2593,#2601,#2631)
  - [Null] Added Vulkan 'window' surface creation via `VK_EXT_headless_surface`
  - [Null] Added EGL context creation on Mesa via `EGL_MESA_platform_surfaceless`

deps/nuklear.h

@@ -423,6 +423,11 @@ NK_STATIC_ASSERT(sizeof(nk_rune) >= 4);
 NK_STATIC_ASSERT(sizeof(nk_size) >= sizeof(void*));
 NK_STATIC_ASSERT(sizeof(nk_ptr) >= sizeof(void*));
 
+#if defined(_MSC_VER)
+/* disable `operands are different enum types` warning on MSVC */ 
+#pragma warning( disable: 5287 )
+#endif
+
 /* ============================================================================
  *
  *                                  API

docs/input.md

@@ -940,9 +940,10 @@ with @ref glfwGetTimerValue.
 uint64_t value = glfwGetTimerValue();
 ```
 
-This value is in 1 / frequency seconds.  The frequency of the raw
+This function returns the current value of the raw timer. To get the current time in 
-timer varies depending on the operating system and hardware.  You can query the
+1 / frequency seconds, use `glfwGetTimerValue() / glfwGetTimerFrequency()`.
-frequency, in Hz, with @ref glfwGetTimerFrequency.
+The frequency of the rawtimer varies depending on the operating system and hardware.
+You can query the frequency, in Hz, with @ref glfwGetTimerFrequency.
 
 ```c
 uint64_t frequency = glfwGetTimerFrequency();

include/GLFW/glfw3.h

@@ -6037,9 +6037,12 @@ GLFWAPI void glfwSetTime(double time);
 
 /*! @brief Returns the current value of the raw timer.
  *
- *  This function returns the current value of the raw timer, measured in
+ *  This function returns the current value of the raw timer. To get the 
- *  1 / frequency seconds.  To get the frequency, call @ref
+ *  current time measured in 1 / frequency seconds, use 
- *  glfwGetTimerFrequency.
+ *  @ref glfwGetTimerValue / @ref glfwGetTimerFrequency.
+ *  To get the frequency, call @ref glfwGetTimerFrequency.
+ *  
+ *  
  *
  *  @return The value of the timer, or zero if an
  *  [error](@ref error_handling) occurred.

src/wl_window.c

@@ -1267,17 +1267,21 @@ static void handleEvents(double* timeout)
 
             if (read(_glfw.wl.keyRepeatTimerfd, &repeats, sizeof(repeats)) == 8)
             {
-                for (uint64_t i = 0; i < repeats; i++)
+                if(_glfw.wl.keyboardFocus)
                 {
-                    _glfwInputKey(_glfw.wl.keyboardFocus,
+                    for (uint64_t i = 0; i < repeats; i++)
-                                  translateKey(_glfw.wl.keyRepeatScancode),
+                    {
-                                  _glfw.wl.keyRepeatScancode,
+                        _glfwInputKey(_glfw.wl.keyboardFocus,
-                                  GLFW_PRESS,
+                                      translateKey(_glfw.wl.keyRepeatScancode),
-                                  _glfw.wl.xkb.modifiers);
+                                      _glfw.wl.keyRepeatScancode,
-                    inputText(_glfw.wl.keyboardFocus, _glfw.wl.keyRepeatScancode);
+                                      GLFW_PRESS,
+                                      _glfw.wl.xkb.modifiers);
+                        inputText(_glfw.wl.keyboardFocus, _glfw.wl.keyRepeatScancode);
+                    }
+
+                    event = GLFW_TRUE;
                 }
 
-                event = GLFW_TRUE;
             }
         }