fixing some minor issues
authorAnselm R. Garbe <arg@suckless.org>
Fri, 05 Jan 2007 16:35:45 +0100
changeset 658 b4d1ef7e407f
parent 657 ce3538c5c0d9
child 659 498b86c72a40
fixing some minor issues
config.mk
view.c
--- a/config.mk	Fri Jan 05 15:37:10 2007 +0100
+++ b/config.mk	Fri Jan 05 16:35:45 2007 +0100
@@ -17,8 +17,8 @@
 # flags
 CFLAGS = -Os ${INCS} -DVERSION=\"${VERSION}\"
 LDFLAGS = ${LIBS}
-#CFLAGS = -g -Wall -O2 ${INCS} -DVERSION=\"${VERSION}\"
-#LDFLAGS = -g ${LIBS}
+CFLAGS = -g -Wall -O2 ${INCS} -DVERSION=\"${VERSION}\"
+LDFLAGS = -g ${LIBS}
 
 # Solaris
 #CFLAGS = -fast ${INCS} -DVERSION=\"${VERSION}\"
--- a/view.c	Fri Jan 05 15:37:10 2007 +0100
+++ b/view.c	Fri Jan 05 16:35:45 2007 +0100
@@ -32,17 +32,17 @@
 static void
 swap(Client *c1, Client *c2) {
 	Client tmp = *c1;
-	Client *cp = c1->prev;
-	Client *cn = c1->next;
+	Client *c1p = c1->prev;
+	Client *c1n = c1->next;
+	Client *c2p = c2->prev;
+	Client *c2n = c2->next;
 
 	*c1 = *c2;
-	c1->prev = cp;
-	c1->next = cn;
-	cp = c2->prev;
-	cn = c2->next;
 	*c2 = tmp;
-	c2->prev = cp;
-	c2->next = cn;
+	c1->prev = c1p;
+	c1->next = c1n;
+	c2->prev = c2p;
+	c2->next = c2n;
 }
 
 static void
@@ -192,10 +192,9 @@
 
 void
 incnmaster(Arg *arg) {
-	if(nmaster + arg->i < 1 || (wah / (nmaster + arg->i) < bh))
+	if((nmaster + arg->i < 1) || (wah / (nmaster + arg->i) < bh))
 		return;
 	nmaster += arg->i;
-
 	arrange();
 }
 
@@ -305,7 +304,9 @@
 		n++;
 
 	c = sel;
-	if(n <= nmaster || (arrange == dofloat))
+	if(arrange == dofloat)
+		return;
+	else if(n <= nmaster)
 		pop(c);
 	else if(ismaster(sel)) {
 		if(!(c = topofstack()))