remove sigchld function

Explicitly setting the handler for SIGCHLD to SIG_IGN tells the OS to
reap zombie processes automatically, which is what we wanted anyway.

1 files changed, 3 insertions, 12 deletions

diff --git a/dwl.c b/dwl.c
index 4098f86..0853f1e 100644
--- a/dwl.c
+++ b/dwl.c
@@ -278,7 +278,6 @@ static void setlayout(const Arg *arg);
 static void setmfact(const Arg *arg);
 static void setmon(Client *c, Monitor *m, unsigned int newtags);
 static void setup(void);
-static void sigchld(int unused);
 static void spawn(const Arg *arg);
 static void tag(const Arg *arg);
 static void tagmon(const Arg *arg);
@@ -1984,8 +1983,9 @@ setup(void)
 	 * clients from the Unix socket, manging Wayland globals, and so on. */
 	dpy = wl_display_create();
 
-	/* clean up child processes immediately */
-	sigchld(0);
+	/* Indicate explicitly to the OS that we are not interested in info
+	 * about child processes (per POSIX.1-2001) */
+	signal(SIGCHLD, SIG_IGN);
 
 	/* The backend is a wlroots feature which abstracts the underlying input and
 	 * output hardware. The autocreate option will choose the most suitable
@@ -2137,15 +2137,6 @@ setup(void)
 }
 
 void
-sigchld(int unused)
-{
-	if (signal(SIGCHLD, sigchld) == SIG_ERR)
-		EBARF("can't install SIGCHLD handler");
-	while (0 < waitpid(-1, NULL, WNOHANG))
-		;
-}
-
-void
 spawn(const Arg *arg)
 {
 	if (fork() == 0) {