Use of _exit() in execunix.c?

From: Craig Rodrigues (rodrigc_at_[hidden])
Date: 2005-07-28 09:59:18

Hi,

Maybe this was there before and I just never paid attention to it.
I recently did a cvs update and compiled bjam
on MacOS X 10.4 and saw this compiler warning:

execunix.c: In function 'execcmd':
execunix.c:153: warning: incompatible implicit declaration of built-in function '_exit'

_exit() is defined in <unistd.h>.
Can we unconditionally include <unistd.h> in this file? <unistd.h>
is a fairly standard Unix header file.

Index: execunix.c
===================================================================
RCS file: /cvsroot/boost/boost/tools/build/jam_src/execunix.c,v
retrieving revision 1.6
diff -u -r1.6 execunix.c
--- execunix.c 12 Feb 2005 02:30:17 -0000 1.6
+++ execunix.c 28 Jul 2005 14:58:43 -0000
@@ -9,9 +9,9 @@
# include "execcmd.h"
# include <errno.h>
# include <time.h>
+# include <unistd.h> /* needed for vfork(), _exit() prototypes */

#if defined(sun) || defined(__sun)
-#include <unistd.h> /* need to include unistd.h on sun for the vfork prototype*/
#include <wait.h>
#endif

-- 
Craig Rodrigues 
rodrigc_at_[hidden]