John> + /* Install a SIGCHLD handler. */ John> + signal (SIGCHLD, sigchld_handler); >> Does this need sigaction + SA_RESTART? >> Otherwise this looks good to me. John> On FreeBSD signal(3) always sets SA_RESTART when installing a handler, John> so this doesn't need an explicit SA_RESTART. Aha, great. Thank you. Tom