# HG changeset patch
# User Anselm R. Garbe <arg@10kloc.org>
# Date 1156858109 -7200
# Node ID 76b62c0c8c117c73fd8694eaca7be7ba3d03b131
# Parent  b00cc483d13b7015d41cb10215c45f5902f355fa
improved selection policy

diff -r b00cc483d13b -r 76b62c0c8c11 view.c
--- a/view.c	Tue Aug 29 13:40:09 2006 +0200
+++ b/view.c	Tue Aug 29 15:28:29 2006 +0200
@@ -3,31 +3,39 @@
  * See LICENSE file for license details.
  */
 #include "dwm.h"
+#include <stdio.h>
 
 /* static */
 
+static Client *
+minclient()
+{
+	Client *c, *min;
+
+	for(min = c = clients; c; c = c->next)
+		if(c->weight < min->weight)
+			min = c;
+	return min;
+}
+
+
 static void
 reorder()
 {
-	Client *c, *orig, *p;
+	Client *c, *newclients, *tail;
 
-	orig = clients;
-	clients = NULL;
-
-	while((c = orig)) {
-		orig = orig->next;
+	newclients = tail = NULL;
+	while((c = minclient())) {
 		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;
+		if(tail) {
+			c->prev = tail;
+			tail->next = c;
+			tail = c;
 		}
 		else
-			clients = c;
+			tail = newclients = c;
 	}
+	clients = newclients;
 }
 
 /* extern */
@@ -59,9 +67,7 @@
 		else
 			ban(c);
 	}
-	if(!sel || !isvisible(sel))
-		sel = getnext(clients);
-	if(sel)
+	if((sel = getnext(clients)))
 		focus(sel);
 	else
 		XSetInputFocus(dpy, root, RevertToPointerRoot, CurrentTime);
@@ -124,9 +130,7 @@
 		else
 			ban(c);
 	}
-	if(!sel || !isvisible(sel))
-		sel = getnext(clients);
-	if(sel)
+	if((sel = getnext(clients)))
 		focus(sel);
 	else
 		XSetInputFocus(dpy, root, RevertToPointerRoot, CurrentTime);