removed getnext/getprev, redundant
authorAnselm R. Garbe <arg@suckless.org>
Fri, 16 Feb 2007 10:20:34 +0100
changeset 761 4e835c2a7a12
parent 760 8ed842c35e8d
child 762 140bcd4782d8
removed getnext/getprev, redundant
dwm.h
tag.c
view.c
--- a/dwm.h	Wed Feb 14 14:01:12 2007 +0100
+++ b/dwm.h	Fri Feb 16 10:20:34 2007 +0100
@@ -128,8 +128,6 @@
 
 /* tag.c */
 extern void initrregs(void);			/* initialize regexps of rules defined in config.h */
-extern Client *getnext(Client *c);		/* returns next visible client */
-extern Client *getprev(Client *c);		/* returns previous visible client */
 extern void settags(Client *c, Client *trans);	/* sets tags of c */
 extern void tag(Arg *arg);			/* tags c with arg's index */
 extern void toggletag(Arg *arg);		/* toggles c tags with arg's index */
--- a/tag.c	Wed Feb 14 14:01:12 2007 +0100
+++ b/tag.c	Fri Feb 16 10:20:34 2007 +0100
@@ -31,18 +31,6 @@
 
 /* extern */
 
-Client *
-getnext(Client *c) {
-	for(; c && !isvisible(c); c = c->next);
-	return c;
-}
-
-Client *
-getprev(Client *c) {
-	for(; c && !isvisible(c); c = c->prev);
-	return c;
-}
-
 void
 initrregs(void) {
 	unsigned int i;
--- a/view.c	Wed Feb 14 14:01:12 2007 +0100
+++ b/view.c	Fri Feb 16 10:20:34 2007 +0100
@@ -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;
 }
 
@@ -125,8 +125,9 @@
    
 	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();
@@ -139,9 +140,10 @@
 
 	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);