RevertToPointerRoot is the worse error case, so use it always

diff --git a/openbox/client.c b/openbox/client.c
index 75ba752b57b8c9d4ef12e028c824b1a989f24184..eddd96c427dc1de5cc08381133ce82aa26071790 100644 (file)
--- a/openbox/client.c
+++ b/openbox/client.c
@@ -1956,9 +1956,10 @@ gboolean client_focus(Client *self)
     }
 
     if (self->can_focus)
-        /* RevertToNone is used so that windows dont get focused inadvertantly
-           by having the pointer in them, and then FocusIn events go missing */
-       XSetInputFocus(ob_display, self->window, RevertToNone,
+        /* RevertToPointerRoot causes much more headache than TevertToNone, so
+           I choose to use it always, hopefully to find errors quicker, if any
+           are left. (I hate X. I hate focus events.) */
+       XSetInputFocus(ob_display, self->window, RevertToPointerRoot,
                        event_lasttime);
 
     if (self->focus_notify) {