don't assert, users can bind stuff however they like. but if they do something that...

[chaz/openbox] / openbox / event.c

diff --git a/openbox/event.c b/openbox/event.c
index 99fa1bf95f18751aebc4ebe8396ae960584d12e9..a2571f0e112dac34c82fca7e537b48b07d003b6b 100644 (file)
--- a/openbox/event.c
+++ b/openbox/event.c
@@ -313,7 +313,7 @@ static gboolean wanted_focusevent(XEvent *e, gboolean in_client_only)
 
         /* It was on a client, was it a valid one?
            It's possible to get a FocusIn event for a client that was managed
-           but has disappeared. Don't even parse those FocusIn events.
+           but has disappeared.
         */
         if (in_client_only) {
             ObWindow *w = g_hash_table_lookup(window_map, &e->xfocus.window);
@@ -394,6 +394,9 @@ static void print_focusevent(XEvent *e)
     case NotifyDetailNone:  detailstr="NotifyDetailNone";  break;
     }
 
+    if (mode == NotifyGrab || mode == NotifyUngrab)
+        return;
+
     g_assert(modestr);
     g_assert(detailstr);
     ob_debug_type(OB_DEBUG_FOCUS, "Focus%s 0x%x mode=%s detail=%s\n",