JasonWoof Got questions, comments, patches, etc.? Contact Jason Woofenden
removed getnext/getprev, redundant
[dwm.git] / view.c
diff --git a/view.c b/view.c
index 8164446..6985dc7 100644 (file)
--- a/view.c
+++ b/view.c
@@ -8,7 +8,7 @@
 
 static Client *
 nexttiled(Client *c) {
-       for(c = getnext(c); c && c->isfloat; c = getnext(c->next));
+       for(; c && (c->isfloat || !isvisible(c)); c = c->next);
        return c;
 }
 
@@ -55,10 +55,12 @@ dofloat(void) {
        Client *c;
 
        for(c = clients; c; c = c->next) {
-               if(isvisible(c))
+               if(isvisible(c)) {
+                       c->isbanned = False;
                        resize(c, True);
+               }
                else
-                       XMoveWindow(dpy, c->win, c->x + 2 * sw, c->y);
+                       ban(c);
        }
        if(!sel || !isvisible(sel)) {
                for(c = stack; c && !isvisible(c); c = c->snext);
@@ -82,6 +84,7 @@ dotile(void) {
 
        for(i = 0, c = clients; c; c = c->next)
                if(isvisible(c)) {
+                       c->isbanned = False;
                        if(c->isfloat) {
                                resize(c, True);
                                continue;
@@ -108,7 +111,7 @@ dotile(void) {
                        i++;
                }
                else
-                       XMoveWindow(dpy, c->win, c->x + 2 * sw, c->y);
+                       ban(c);
        if(!sel || !isvisible(sel)) {
                for(c = stack; c && !isvisible(c); c = c->snext);
                focus(c);
@@ -122,8 +125,9 @@ focusnext(Arg *arg) {
    
        if(!sel)
                return;
-       if(!(c = getnext(sel->next)))
-               c = getnext(clients);
+       for(c = sel->next; c && !isvisible(c); c = c->next);
+       if(!c)
+               for(c = clients; c && !isvisible(c); c = c->next);
        if(c) {
                focus(c);
                restack();
@@ -136,9 +140,10 @@ focusprev(Arg *arg) {
 
        if(!sel)
                return;
-       if(!(c = getprev(sel->prev))) {
+       for(c = sel->prev; c && !isvisible(c); c = c->prev);
+       if(!c) {
                for(c = clients; c && c->next; c = c->next);
-               c = getprev(c);
+               for(; c && !isvisible(c); c = c->prev);
        }
        if(c) {
                focus(c);