From: Dana Jansens <danakj@orodu.net>
Date: Fri, 28 Mar 2003 10:47:48 +0000 (+0000)
Subject: RevertToPointerRoot is the worse error case, so use it always
X-Git-Url: https://git.brokenzipper.com/gitweb?a=commitdiff_plain;h=0fcd4c30ccd9a857d3ed643abf0c3463d1d5c8d7;p=chaz%2Fopenbox

RevertToPointerRoot is the worse error case, so use it always
---

diff --git a/openbox/client.c b/openbox/client.c
index 75ba752b..eddd96c4 100644
--- 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) {