Amebis/wxWidgets
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Removed wxLogSysError call in wxHandleProcessTermination, and ensured

Browse Source 
the cleanup always happens. I hope this is right... but if processes
are killed elsewhere, we don't want this function to generate an error
and not do the correct cleanup.


git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@11510 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
This commit is contained in:
Julian Smart
2001-08-29 11:24:50 +00:00
parent e126b44d22
commit 09d68ec1c3
1 changed files with 7 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
src/unix/utilsunx.cpp
View File

@@ -301,14 +301,19 @@ void wxHandleProcessTermination(wxEndProcessData *proc_data)
if( rc == -1 || ! (WIFEXITED(status) || WIFSIGNALED(status)) ) if( rc == -1 || ! (WIFEXITED(status) || WIFSIGNALED(status)) )
{ {
wxLogSysError(_("Waiting for subprocess termination failed")); // wxLogSysError(_("Waiting for subprocess termination failed"));
/* AFAIK, this can only happen if something went wrong within /* AFAIK, this can only happen if something went wrong within
wxGTK, i.e. due to a race condition or some serious bug. wxGTK, i.e. due to a race condition or some serious bug.
After having fixed the order of statements in After having fixed the order of statements in
GTK_EndProcessDetector(). (KB) GTK_EndProcessDetector(). (KB)
JACS adds -- I have other code that kills a process recursively
and calls waitpid; so this function then generates an error.
I've commented out the wxLogSysError and the 'else' so that
termination is always done properly.
*/ */
} }
else // else
{ {
// notify user about termination if required // notify user about termination if required
if (proc_data->process) if (proc_data->process)