X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/14f355c2b5c71fc7c3d680aea366582d2ac60f7b..3e4e0908a7c8217f87d1c7a521f590c1de3bc0f9:/src/common/process.cpp?ds=sidebyside diff --git a/src/common/process.cpp b/src/common/process.cpp index 6e6a931c2f..210068e955 100644 --- a/src/common/process.cpp +++ b/src/common/process.cpp @@ -1,5 +1,5 @@ ///////////////////////////////////////////////////////////////////////////// -// Name: process.cpp +// Name: src/common/process.cpp // Purpose: Process termination classes // Author: Guilhem Lavaux // Modified by: Vadim Zeitlin to check error codes, added Detach() method @@ -17,10 +17,6 @@ // headers // ---------------------------------------------------------------------------- -#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA) - #pragma implementation "process.h" -#endif - // For compilers that support precompilation, includes "wx.h". #include "wx/wxprec.h" @@ -34,7 +30,7 @@ // event tables and such // ---------------------------------------------------------------------------- -DEFINE_EVENT_TYPE(wxEVT_END_PROCESS) +wxDEFINE_EVENT( wxEVT_END_PROCESS, wxProcessEvent ); IMPLEMENT_DYNAMIC_CLASS(wxProcess, wxEvtHandler) IMPLEMENT_DYNAMIC_CLASS(wxProcessEvent, wxEvent) @@ -53,6 +49,7 @@ void wxProcess::Init(wxEvtHandler *parent, int id, int flags) SetNextHandler(parent); m_id = id; + m_pid = 0; m_redirect = (flags & wxPROCESS_REDIRECT) != 0; #if wxUSE_STREAMS @@ -67,13 +64,16 @@ wxProcess *wxProcess::Open(const wxString& cmd, int flags) { wxASSERT_MSG( !(flags & wxEXEC_SYNC), wxT("wxEXEC_SYNC should not be used." )); wxProcess *process = new wxProcess(wxPROCESS_REDIRECT); - if ( !wxExecute(cmd, flags, process) ) + long pid = wxExecute(cmd, flags, process); + if( !pid ) { // couldn't launch the process delete process; return NULL; } + process->SetPid(pid); + return process; } @@ -102,7 +102,13 @@ void wxProcess::OnTerminate(int pid, int status) void wxProcess::Detach() { - SetNextHandler(NULL); + // we just detach from the next handler of the chain (i.e. our "parent" -- see ctor) + // not also from the previous handler like wxEvtHandler::Unlink() would do: + + if (m_nextHandler) + m_nextHandler->SetPreviousHandler(m_previousHandler); + + m_nextHandler = NULL; } // ---------------------------------------------------------------------------- @@ -142,10 +148,10 @@ bool wxProcess::IsErrorAvailable() const // ---------------------------------------------------------------------------- /* static */ -wxKillError wxProcess::Kill(int pid, wxSignal sig) +wxKillError wxProcess::Kill(int pid, wxSignal sig, int flags) { wxKillError rc; - (void)wxKill(pid, sig, &rc); + (void)wxKill(pid, sig, &rc, flags); return rc; } @@ -157,16 +163,16 @@ bool wxProcess::Exists(int pid) { case wxKILL_OK: case wxKILL_ACCESS_DENIED: - return TRUE; + return true; default: case wxKILL_ERROR: case wxKILL_BAD_SIGNAL: - wxFAIL_MSG( _T("unexpected wxProcess::Kill() return code") ); + wxFAIL_MSG( wxT("unexpected wxProcess::Kill() return code") ); // fall through case wxKILL_NO_PROCESS: - return FALSE; + return false; } }