X-Git-Url: https://git.dogcows.com/gitweb?a=blobdiff_plain;f=openbox%2Fgrab.c;h=97a1a3db2acb631f86351fc16b59bbd64b5ced48;hb=9bae1e3128d75ff0768bfb78a1437dec6e6f1173;hp=ab38f27b826f83192fd81169b6e31a21a4abb709;hpb=1d5fa24e4b92fbd2fc16847612a6825ca08d6567;p=chaz%2Fopenbox

diff --git a/openbox/grab.c b/openbox/grab.c
index ab38f27b..97a1a3db 100644
--- a/openbox/grab.c
+++ b/openbox/grab.c
@@ -1,5 +1,6 @@
 #include "openbox.h"
 #include "event.h"
+#include "xerror.h"
 
 #include <glib.h>
 #include <X11/Xlib.h>
@@ -14,18 +15,12 @@ static unsigned int mask_list[MASK_LIST_SIZE];
 void grab_keyboard(gboolean grab)
 {
     if (grab) {
-        if (kgrabs++ == 0) {
-            g_message("GRABBING KEYBOARD %d", kgrabs);
+        if (kgrabs++ == 0)
             XGrabKeyboard(ob_display, ob_root, 0, GrabModeAsync, GrabModeSync,
-                          CurrentTime);
-        } else
-            g_message("NOT GRABBING KEYBOARD %d", kgrabs);
+                          event_lasttime);
     } else if (kgrabs > 0) {
-        if (--kgrabs == 0) {
-            g_message("UNGRABBING KEYBOARD %d", kgrabs);
-            XUngrabKeyboard(ob_display, CurrentTime);
-        } else
-            g_message("NOT UNGRABBING KEYBOARD %d", kgrabs);
+        if (--kgrabs == 0)
+            XUngrabKeyboard(ob_display, event_lasttime);
     }
 }
 
@@ -34,10 +29,10 @@ void grab_pointer(gboolean grab, Cursor cur)
     if (grab) {
         if (pgrabs++ == 0)
             XGrabPointer(ob_display, ob_root, False, 0, GrabModeAsync,
-                         GrabModeAsync, FALSE, cur, CurrentTime);
+                         GrabModeAsync, FALSE, cur, event_lasttime);
     } else if (pgrabs > 0) {
         if (--pgrabs == 0)
-            XUngrabPointer(ob_display, CurrentTime);
+            XUngrabPointer(ob_display, event_lasttime);
     }
 }
 
@@ -87,7 +82,7 @@ void grab_button(guint button, guint state, Window win, guint mask,
 
     for (i = 0; i < MASK_LIST_SIZE; ++i)
         XGrabButton(ob_display, button, state | mask_list[i], win, FALSE, mask,
-                    pointer_mode, GrabModeAsync, None, None);
+                    pointer_mode, GrabModeSync, None, None);
 }
 
 void ungrab_button(guint button, guint state, Window win)
@@ -102,9 +97,14 @@ void grab_key(guint keycode, guint state, int keyboard_mode)
 {
     guint i;
 
+    xerror_set_ignore(TRUE); /* can get BadAccess' from these */
+    xerror_occured = FALSE;
     for (i = 0; i < MASK_LIST_SIZE; ++i)
         XGrabKey(ob_display, keycode, state | mask_list[i], ob_root, FALSE,
-                 GrabModeAsync, keyboard_mode);
+                 GrabModeSync, keyboard_mode);
+    xerror_set_ignore(FALSE);
+    if (xerror_occured)
+        g_warning("failed to grab keycode %d modifiers %d", keycode, state);
 }
 
 void ungrab_all_keys()