Forward declare Xkb in keyboard_input.h

autotests/integration/globalshortcuts_test.cpp

@@ -19,6 +19,7 @@
 #include "wayland_server.h"
 #include "workspace.h"
 #include "x11window.h"
+#include "xkb.h"
 
 #include <KWayland/Client/surface.h>
 

autotests/integration/keyboard_layout_test.cpp

@@ -15,6 +15,7 @@
 #include "wayland_server.h"
 #include "window.h"
 #include "workspace.h"
+#include "xkb.h"
 
 #include <KConfigGroup>
 #include <KGlobalAccel>

autotests/integration/modifier_only_shortcut_test.cpp

@@ -16,6 +16,7 @@
 #include "keyboard_input.h"
 #include "wayland_server.h"
 #include "workspace.h"
+#include "xkb.h"
 
 #include <KConfigGroup>
 

src/backends/x11/standalone/x11_standalone_platform.cpp

@@ -39,6 +39,7 @@
 #include "x11_standalone_output.h"
 #include "x11_standalone_overlaywindow.h"
 #include "x11_standalone_screenedges_filter.h"
+#include "xkb.h"
 
 #include "../common/kwinxrenderutils.h"
 

src/debug_console.cpp

@@ -31,6 +31,7 @@
 #include "waylandwindow.h"
 #include "workspace.h"
 #include "x11window.h"
+#include "xkb.h"
 #include <cerrno>
 #include <kwinglplatform.h>
 #include <kwinglutils.h>

src/input.cpp

@@ -49,6 +49,7 @@
 #include "wayland/tablet_v2_interface.h"
 #include "wayland_server.h"
 #include "workspace.h"
+#include "xkb.h"
 #include "xwayland/xwayland_interface.h"
 
 #include <KDecoration2/Decoration>

src/inputmethod.cpp

@@ -30,6 +30,7 @@
 #include "wayland/seat_interface.h"
 #include "wayland/surface_interface.h"
 #include "wayland/textinput_v3_interface.h"
+#include "xkb.h"
 
 #include <KLocalizedString>
 #include <KShell>

src/keyboard_input.cpp

@@ -23,6 +23,7 @@
 #include "wayland_server.h"
 #include "window.h"
 #include "workspace.h"
+#include "xkb.h"
 // screenlocker
 #if KWIN_BUILD_SCREENLOCKER
 #include <KScreenLocker/KsldApp>
@@ -50,6 +51,21 @@ KeyboardInputRedirection::KeyboardInputRedirection(InputRedirection *parent)
 
 KeyboardInputRedirection::~KeyboardInputRedirection() = default;
 
+Xkb *KeyboardInputRedirection::xkb() const
+{
+    return m_xkb.get();
+}
+
+Qt::KeyboardModifiers KeyboardInputRedirection::modifiers() const
+{
+    return m_xkb->modifiers();
+}
+
+Qt::KeyboardModifiers KeyboardInputRedirection::modifiersRelevantForGlobalShortcuts() const
+{
+    return m_xkb->modifiersRelevantForGlobalShortcuts();
+}
+
 class KeyStateChangedSpy : public InputEventSpy
 {
 public:

src/keyboard_input.h

@@ -10,7 +10,6 @@
 #define KWIN_KEYBOARD_INPUT_H
 
 #include "input.h"
-#include "xkb.h"
 
 #include <QObject>
 #include <QPointF>
@@ -37,6 +36,7 @@ class InputDevice;
 class InputRedirection;
 class KeyboardLayout;
 class ModifiersChangedSpy;
+class Xkb;
 
 class KWIN_EXPORT KeyboardInputRedirection : public QObject
 {
@@ -63,18 +63,9 @@ public:
      */
     void processKeymapChange(int fd, uint32_t size);
 
-    Xkb *xkb() const
+    Xkb *xkb() const;
-    {
+    Qt::KeyboardModifiers modifiers() const;
-        return m_xkb.get();
+    Qt::KeyboardModifiers modifiersRelevantForGlobalShortcuts() const;
-    }
-    Qt::KeyboardModifiers modifiers() const
-    {
-        return m_xkb->modifiers();
-    }
-    Qt::KeyboardModifiers modifiersRelevantForGlobalShortcuts() const
-    {
-        return m_xkb->modifiersRelevantForGlobalShortcuts();
-    }
 
 Q_SIGNALS:
     void ledsChanged(KWin::LEDs);

src/keyboard_layout.cpp

@@ -10,6 +10,7 @@
 #include "input_event.h"
 #include "keyboard_input.h"
 #include "keyboard_layout_switching.h"
+#include "xkb.h"
 
 #include <KGlobalAccel>
 #include <KLocalizedString>

src/keyboard_repeat.cpp

@@ -12,6 +12,7 @@
 #include "wayland/keyboard_interface.h"
 #include "wayland/seat_interface.h"
 #include "wayland_server.h"
+#include "xkb.h"
 
 #include <QTimer>
 

src/plugins/buttonrebinds/buttonrebindsfilter.cpp

@@ -9,6 +9,7 @@
 
 #include "input_event.h"
 #include "keyboard_input.h"
+#include "xkb.h"
 
 #include <KKeyServer>