this version should also work with cornercases (like unmanage during !issel, etc.)

diff --git a/client.c b/client.c
index 5309d0eb9b44c3bbc8fa1d221110c4b340cbfa0a..5c69718257ae0990c7f887ed6e2aac82677dc5ab 100644 (file)
--- a/client.c
+++ b/client.c
@@ -79,15 +79,16 @@ configure(Client *c) {
 
 void
 focus(Client *c) {
-       Client *old = sel;
-
        if(c && !isvisible(c))
                return;
 
-       if(old && old != c) {
-               grabbuttons(old, False);
-               XSetWindowBorder(dpy, old->win, dc.norm[ColBorder]);
+       if(sel && sel != c) {
+               grabbuttons(sel, False);
+               XSetWindowBorder(dpy, sel->win, dc.norm[ColBorder]);
        }
+       sel = c;
+       if(!issel)
+               return;
        if(c) {
                detachstack(c);
                c->snext = stack;
@@ -96,9 +97,8 @@ focus(Client *c) {
                XSetWindowBorder(dpy, c->win, dc.sel[ColBorder]);
                XSetInputFocus(dpy, c->win, RevertToPointerRoot, CurrentTime);
        }
-       else if(issel)
+       else
                XSetInputFocus(dpy, root, RevertToPointerRoot, CurrentTime);
-       sel = c;
        drawstatus();
 }