diff --git a/winsup/cygwin/ChangeLog b/winsup/cygwin/ChangeLog index afc5ed203..917bf256d 100644 --- a/winsup/cygwin/ChangeLog +++ b/winsup/cygwin/ChangeLog @@ -1,3 +1,7 @@ +2011-11-14 Christopher Faylor + + * pinfo.cc (_pinfo::dup_proc_pipe): Fatalize a warning when debugging. + 2011-11-13 Christopher Faylor Remove erroneously checked-in debugging statements. diff --git a/winsup/cygwin/pinfo.cc b/winsup/cygwin/pinfo.cc index 52aac9349..8c44fc5ad 100644 --- a/winsup/cygwin/pinfo.cc +++ b/winsup/cygwin/pinfo.cc @@ -989,6 +989,11 @@ proc_waiter (void *arg) return 0; } +#ifdef DEBUGGING +#define warn_printf api_fatal +#else +#define warn_printf system_printf +#endif HANDLE _pinfo::dup_proc_pipe (HANDLE hProcess) { @@ -1004,8 +1009,8 @@ _pinfo::dup_proc_pipe (HANDLE hProcess) if (!res && WaitForSingleObject (hProcess, 0) != WAIT_OBJECT_0) { wr_proc_pipe = orig_wr_proc_pipe; - system_printf ("DuplicateHandle failed, pid %d, hProcess %p, wr_proc_pipe %p, %E", - pid, hProcess, wr_proc_pipe); + warn_printf ("something failed for pid %d: res %d, hProcess %p, wr_proc_pipe %p vs. %p, %E", + res, pid, hProcess, wr_proc_pipe, orig_wr_proc_pipe); } else {