Made slop be really aggressive with mouse and keyboard grabs. Failing to grab the mouse results in a fatal shutdown now.

CMakeLists.txt

@@ -21,7 +21,7 @@ endif()
 
 include_directories("${PROJECT_BINARY_DIR}")
 
-add_definitions(-DSLOP_VERSION="v5.3.31")
+add_definitions(-DSLOP_VERSION="v5.3.32")
 
 # The names have to be unique unfortunately.
 set(EXECUTABLE_NAME "slop")

src/keyboard.cpp

@@ -1,3 +1,5 @@
+#include <chrono>
+#include <thread>
 #include "keyboard.hpp"
 
 bool slop::Keyboard::getKey( KeySym key ) {
@@ -41,8 +43,15 @@ void slop::Keyboard::update() {
 slop::Keyboard::Keyboard( X11* x11 ) {
     this->x11 = x11;
     int err = XGrabKeyboard( x11->display, x11->root, False, GrabModeAsync, GrabModeAsync, CurrentTime );
+    int tries = 0;
+    while( err != GrabSuccess && tries < 5 ) {
+        std::this_thread::sleep_for(std::chrono::milliseconds(100));
+        err = XGrabKeyboard( x11->display, x11->root, False, GrabModeAsync, GrabModeAsync, CurrentTime );
+        tries++;
+    }
+    // Non-fatal.
     if ( err != GrabSuccess ) {
-        //throw new std::runtime_error( "Failed to grab keyboard.\n" );
+        //throw new std::runtime_error( "Couldn't grab the keyboard after 10 tries." );
     }
     XQueryKeymap( x11->display, deltaState );
     keyDown = false;

src/mouse.cpp

@@ -1,3 +1,5 @@
+#include <chrono>
+#include <thread>
 #include "mouse.hpp"
 
 void slop::Mouse::setButton( int button, int state ) {
@@ -44,9 +46,20 @@ slop::Mouse::Mouse(X11* x11, int nodecorations, Window ignoreWindow ) {
     currentCursor = XC_cross;
     xcursor = XCreateFontCursor( x11->display, XC_cross );
     hoverWindow = None;
-    XGrabPointer( x11->display, x11->root, True,
-                  PointerMotionMask | ButtonPressMask | ButtonReleaseMask | EnterWindowMask,
-                  GrabModeAsync, GrabModeAsync, None, xcursor, CurrentTime );
+    int err = XGrabPointer( x11->display, x11->root, True,
+                            PointerMotionMask | ButtonPressMask | ButtonReleaseMask | EnterWindowMask,
+                            GrabModeAsync, GrabModeAsync, None, xcursor, CurrentTime );
+    int tries = 0;
+    while( err != GrabSuccess && tries < 5 ) {
+        std::this_thread::sleep_for(std::chrono::milliseconds(100));
+        err = XGrabPointer( x11->display, x11->root, True,
+                                PointerMotionMask | ButtonPressMask | ButtonReleaseMask | EnterWindowMask,
+                                GrabModeAsync, GrabModeAsync, None, xcursor, CurrentTime );
+        tries++;
+    }
+    if ( err != GrabSuccess ) {
+        throw new std::runtime_error( "Couldn't grab the mouse after 10 tries." );
+    }
     this->nodecorations = nodecorations;
     this->ignoreWindow = ignoreWindow;
     hoverWindow = findWindow(x11->root);