diff --git a/winsup/cygwin/ChangeLog b/winsup/cygwin/ChangeLog index 062c6357f..057318949 100644 --- a/winsup/cygwin/ChangeLog +++ b/winsup/cygwin/ChangeLog @@ -1,3 +1,8 @@ +2003-12-07 Christopher Faylor + + * spawn.cc (spawn_guts): Change type back to 'int' after erroneous + change below. + 2003-12-07 Christopher Faylor Change use of BOOL, TRUE, FALSE to bool, true, false, as appropriate, diff --git a/winsup/cygwin/fhandler_tty.cc b/winsup/cygwin/fhandler_tty.cc index a27ee5827..c5718a380 100644 --- a/winsup/cygwin/fhandler_tty.cc +++ b/winsup/cygwin/fhandler_tty.cc @@ -1306,15 +1306,6 @@ fhandler_pty_master::ptsname () void fhandler_tty_common::set_close_on_exec (int val) { -#ifndef DEBUGGING - fhandler_base::set_close_on_exec (val); -#else - /* FIXME: This is a duplication from fhandler_base::set_close_on_exec. - It is here because we need to specify the "from_pty" stuff here or - we'll get warnings from ForceCloseHandle when debugging. */ - set_inheritance (get_io_handle (), val); - set_close_on_exec_flag (val); -#endif if (get_major () == DEV_TTYS_MAJOR && get_io_handle () == cygheap->ctty.get_io_handle ()) set_close_on_exec_flag (val); @@ -1332,6 +1323,15 @@ fhandler_tty_common::set_close_on_exec (int val) set_inheritance (input_mutex, val); set_inheritance (input_available_event, val); set_inheritance (output_handle, val); +#ifndef DEBUGGING + fhandler_base::set_close_on_exec (val); +#else + /* FIXME: This is a duplication from fhandler_base::set_close_on_exec. + It is here because we need to specify the "from_pty" stuff here or + we'll get warnings from ForceCloseHandle when debugging. */ + set_inheritance (get_io_handle (), val); + set_close_on_exec_flag (val); +#endif } } diff --git a/winsup/cygwin/spawn.cc b/winsup/cygwin/spawn.cc index 11a3616c1..0e288dfa6 100644 --- a/winsup/cygwin/spawn.cc +++ b/winsup/cygwin/spawn.cc @@ -345,7 +345,7 @@ do_cleanup (void *args) } -static bool __stdcall +static int __stdcall spawn_guts (const char * prog_arg, const char *const *argv, const char *const envp[], int mode) { diff --git a/winsup/cygwin/wait.cc b/winsup/cygwin/wait.cc index 79c8bf64a..d3c3416ee 100644 --- a/winsup/cygwin/wait.cc +++ b/winsup/cygwin/wait.cc @@ -48,14 +48,13 @@ wait4 (int intpid, int *status, int options, struct rusage *r) int res; waitq *w; HANDLE waitfor; - bool sawsig; pthread_testcancel (); while (1) { sig_dispatch_pending (); - sawsig = 0; + bool sawsig = false; if (options & ~(WNOHANG | WUNTRACED)) { set_errno (EINVAL); @@ -73,7 +72,7 @@ wait4 (int intpid, int *status, int options, struct rusage *r) w->rusage = r; sigproc_printf ("calling proc_subproc, pid %d, options %d", w->pid, w->options); - if (!proc_subproc (PROC_WAIT, (DWORD)w)) + if (!proc_subproc (PROC_WAIT, (DWORD) w)) { set_errno (ENOSYS); paranoid_printf ("proc_subproc returned 0"); @@ -101,7 +100,7 @@ wait4 (int intpid, int *status, int options, struct rusage *r) { call_signal_handler_now (); set_sig_errno (EINTR); - sawsig = 1; + sawsig = true; res = -1; } else if (res != WAIT_OBJECT_0)