about summary refs log tree commit diff stats
path: root/dwm.c
diff options
context:
space:
mode:
authora@null <unknown>2008-12-12 19:55:03 +0000
committera@null <unknown>2008-12-12 19:55:03 +0000
commit62a491e36d493bf4585b348a4c28c5ed9bd3ea4e (patch)
treea3ced66b4461df644560c74e258eded2f6b9045f /dwm.c
parent0b5dcf229f64a23438e6ef5929d78b7f9bd48355 (diff)
downloaddwm-62a491e36d493bf4585b348a4c28c5ed9bd3ea4e.tar.gz
re-applied Neale's spawn patch, credited Neale in LICENSE
Diffstat (limited to 'dwm.c')
-rw-r--r--dwm.c25
1 files changed, 14 insertions, 11 deletions
diff --git a/dwm.c b/dwm.c
index dc63a0b..d6dc467 100644
--- a/dwm.c
+++ b/dwm.c
@@ -178,6 +178,7 @@ static void setlayout(const Arg *arg);
 static void setmfact(const Arg *arg);
 static void setup(void);
 static void showhide(Client *c);
+static void sigchld(int signal);
 static void spawn(const Arg *arg);
 static void tag(const Arg *arg);
 static int textnw(const char *text, unsigned int len);
@@ -1340,22 +1341,24 @@ showhide(Client *c) {
 	}
 }
 
+
+void
+sigchld(int signal) {
+	while(0 < waitpid(-1, NULL, WNOHANG));
+}
+
 void
 spawn(const Arg *arg) {
-	/* The double-fork construct avoids zombie processes and keeps the code
-	 * clean from stupid signal handlers. */
+	signal(SIGCHLD, sigchld);
 	if(fork() == 0) {
-		if(fork() == 0) {
-			if(dpy)
-				close(ConnectionNumber(dpy));
-			setsid();
-			execvp(((char **)arg->v)[0], (char **)arg->v);
-			fprintf(stderr, "dwm: execvp %s", ((char **)arg->v)[0]);
-			perror(" failed");
-		}
+		if(dpy)
+			close(ConnectionNumber(dpy));
+		setsid();
+		execvp(((char **)arg->v)[0], (char **)arg->v);
+		fprintf(stderr, "dwm: execvp %s", ((char **)arg->v)[0]);
+		perror(" failed");
 		exit(0);
 	}
-	wait(0);
 }
 
 void