still something wrong with reorder()
authorAnselm R. Garbe <arg@10kloc.org>
Tue, 29 Aug 2006 13:40:09 +0200
changeset 381 b00cc483d13b
parent 380 4bf79305d675
child 382 76b62c0c8c11
still something wrong with reorder()
client.c
dwm.h
tag.c
view.c
--- a/client.c	Tue Aug 29 09:57:57 2006 +0200
+++ b/client.c	Tue Aug 29 13:40:09 2006 +0200
@@ -241,7 +241,10 @@
 			|| (c->maxw && c->minw &&
 				c->maxw == c->minw && c->maxh == c->minh);
 
-	attach(c);
+	if(clients)
+		clients->prev = c;
+	c->next = clients;
+	clients = c;
 
 	settitle(c);
 	if(isvisible(c))
--- a/dwm.h	Tue Aug 29 09:57:57 2006 +0200
+++ b/dwm.h	Tue Aug 29 13:40:09 2006 +0200
@@ -56,7 +56,7 @@
 	int basew, baseh, incw, inch, maxw, maxh, minw, minh;
 	int grav;
 	long flags; 
-	unsigned int border;
+	unsigned int border, weight;
 	Bool isfloat;
 	Bool ismax;
 	Bool *tags;
@@ -127,7 +127,6 @@
 extern void spawn(Arg *arg);
 
 /* view.c */
-extern void attach(Client *c);
 extern void detach(Client *c);
 extern void dofloat(Arg *arg);
 extern void dotile(Arg *arg);
--- a/tag.c	Tue Aug 29 09:57:57 2006 +0200
+++ b/tag.c	Tue Aug 29 13:40:09 2006 +0200
@@ -106,6 +106,8 @@
 	if(!matched)
 		for(i = 0; i < ntags; i++)
 			c->tags[i] = seltag[i];
+	for(i = 0; i < ntags && !c->tags[i]; i++);
+	c->weight = i;
 }
 
 void
@@ -120,8 +122,6 @@
 		sel->tags[i] = False;
 	sel->tags[arg->i] = True;
 	settitle(sel);
-	detach(sel);
-	attach(sel);
 	if(!isvisible(sel))
 		arrange(NULL);
 	else
@@ -141,8 +141,6 @@
 	if(i == ntags)
 		sel->tags[arg->i] = True;
 	settitle(sel);
-	detach(sel);
-	attach(sel);
 	if(!isvisible(sel))
 		arrange(NULL);
 	else
--- a/view.c	Tue Aug 29 09:57:57 2006 +0200
+++ b/view.c	Tue Aug 29 13:40:09 2006 +0200
@@ -6,27 +6,28 @@
 
 /* static */
 
-static Client *
-getslot(Client *c)
+static void
+reorder()
 {
-	unsigned int i, tic;
-	Client *p;
+	Client *c, *orig, *p;
+
+	orig = clients;
+	clients = NULL;
 
-	for(tic = 0; tic < ntags && !c->tags[tic]; tic++);
-	for(p = clients; p; p = p->next) {
-		for(i = 0; i < ntags && !p->tags[i]; i++);
-		if(tic < i)
-			return p;
+	while((c = orig)) {
+		orig = orig->next;
+		detach(c);
+
+		for(p = clients; p && p->next && p->weight <= c->weight; p = p->next);
+		c->prev = p;
+		if(p) {
+			if((c->next = p->next))
+				c->next->prev = c;
+			p->next = c;
+		}
+		else
+			clients = c;
 	}
-	return p;
-}
-
-static Client *
-tail()
-{
-	Client *c;
-	for(c = clients; c && c->next; c = c->next);
-	return c;
 }
 
 /* extern */
@@ -34,35 +35,6 @@
 void (*arrange)(Arg *) = DEFMODE;
 
 void
-attach(Client *c)
-{
-	Client *p;
-
-	if(!clients) {
-		clients = c;
-		return;
-	}
-	if(!(p = getnext(clients)) && !(p = getslot(c))) {
-		p = tail();
-		c->prev = p;
-		p->next = c;
-		return;
-	}
-
-	if(p == clients) {
-		c->next = clients;
-		clients->prev = c;
-		clients = c;
-	}
-	else {
-		p->prev->next = c;
-		c->prev = p->prev;
-		p->prev = c;
-		c->next = p;
-	}
-}
-
-void
 detach(Client *c)
 {
 	if(c->prev)
@@ -277,6 +249,7 @@
 	for(i = 0; i < ntags && !seltag[i]; i++);
 	if(i == ntags)
 		seltag[arg->i] = True; /* cannot toggle last view */
+	reorder();
 	arrange(NULL);
 }
 
@@ -284,10 +257,12 @@
 view(Arg *arg)
 {
 	unsigned int i;
+	Client *c;
 
 	for(i = 0; i < ntags; i++)
 		seltag[i] = False;
 	seltag[arg->i] = True;
+	reorder();
 	arrange(NULL);
 }
 
@@ -303,7 +278,9 @@
 		if(!(c = getnext(c->next)))
 			return;
 	detach(c);
-	attach(c);
+	c->next = clients;
+	clients->prev = c;
+	clients = c;
 	focus(c);
 	arrange(NULL);
 }