1. corrected (but the fix is ugly) the multiple def button problem

[wxWidgets.git] / src / common / process.cpp

diff --git a/src/common/process.cpp b/src/common/process.cpp
index d1c255ac95bc639c79543575a15074386234a162..6440993d20363b7011338c325bab4888058e00ec 100644 (file)
--- a/src/common/process.cpp
+++ b/src/common/process.cpp
@@ -2,56 +2,52 @@
 // Name:        process.cpp
 // Purpose:     Process termination classes
 // Author:      Guilhem Lavaux
-// Modified by:
+// Modified by: Vadim Zeitlin to check error codes, added Detach() method
 // Created:     24/06/98
 // RCS-ID:      $Id$
 // Copyright:   (c) Guilhem Lavaux
-// Licence:    wxWindows license
+// Licence:     wxWindows license
 /////////////////////////////////////////////////////////////////////////////
 
 #ifdef __GNUG__
-#pragma implementation "process.h"
+    #pragma implementation "process.h"
 #endif
 
 // For compilers that support precompilation, includes "wx.h".
 #include "wx/wxprec.h"
 
 #ifdef __BORLANDC__
-#pragma hdrstop
+    #pragma hdrstop
 #endif
 
 #ifndef WX_PRECOMP
-#include "wx/defs.h"
+    #include "wx/defs.h"
 #endif
 
 #include "wx/process.h"
 
-#if !USE_SHARED_LIBRARY
 IMPLEMENT_DYNAMIC_CLASS(wxProcess, wxEvtHandler)
 IMPLEMENT_DYNAMIC_CLASS(wxProcessEvent, wxEvent)
-#endif
 
 wxProcess::wxProcess(wxEvtHandler *parent, int id)
 {
-  if (parent)
-    SetPreviousHandler(parent);
+    if (parent)
+        SetNextHandler(parent);
 
-  m_id = id;
+    m_id = id;
 }
 
-wxProcess::~wxProcess()
+void wxProcess::OnTerminate(int pid, int status)
 {
-}
+    wxProcessEvent event(m_id, pid, status);
 
-void wxProcess::OnTerminate(int pid)
-{
-  wxProcessEvent event(m_id, pid);
-
-  ProcessEvent(event);
+    if ( !ProcessEvent(event) )
+        delete this;
+    //else: the object which processed the event is responsible for deleting
+    //      us!
 }
 
-wxProcessEvent::wxProcessEvent(int id, int pid):
-  wxEvent(id), m_pid(pid)
+void wxProcess::Detach()
 {
+    SetNextHandler(NULL);
 }
-