As i dive throught the dwm code, i saw that my previous un/ban was
somewhat incomplete as there are some places where ban() can be used and
it isn't. Patch attached.

-- 
Premysl "Anydot" Hruby >> http://na.srck.net <<
diff -r 2e7a0fe53e4e client.c
--- a/client.c  Sun Jun 03 11:33:03 2007 +0200
+++ b/client.c  Sun Jun 03 11:34:01 2007 +0200
@@ -216,8 +216,7 @@ manage(Window w, XWindowAttributes *wa) 
                c->isfloating = (rettrans == Success) || c->isfixed;
        attach(c);
        attachstack(c);
-       c->isbanned = True;
-       XMoveWindow(dpy, w, c->x + 2 * sw, c->y);
+       ban(c);
        XMapWindow(dpy, w);
        setclientstate(c, NormalState);
        focus(c);
diff -r 2e7a0fe53e4e layout.c
--- a/layout.c  Sun Jun 03 11:33:03 2007 +0200
+++ b/layout.c  Sun Jun 03 11:35:21 2007 +0200
@@ -103,9 +103,7 @@ floating(void) {
 
        for(c = clients; c; c = c->next)
                if(isvisible(c)) {
-                       if(c->isbanned)
-                               XMoveWindow(dpy, c->win, c->x, c->y);
-                       c->isbanned = False;
+                       unban(c);
                        resize(c, c->x, c->y, c->w, c->h, True);
                }
                else
diff -r 2e7a0fe53e4e main.c
--- a/main.c    Sun Jun 03 11:33:03 2007 +0200
+++ b/main.c    Sun Jun 03 11:35:49 2007 +0200
@@ -40,8 +40,7 @@ cleanup(void) {
 cleanup(void) {
        close(STDIN_FILENO);
        while(stack) {
-               if(stack->isbanned)
-                       XMoveWindow(dpy, stack->win, stack->x, stack->y);
+               unban(stack);
                unmanage(stack);
        }
        if(dc.font.set)

Reply via email to