[ 1508778 ] Fix for wxOwnerDrawnComboBox list selection rendering.

[wxWidgets.git] / src / msw / thread.cpp

diff --git a/src/msw/thread.cpp b/src/msw/thread.cpp
index 9a0bdbb46cf4ee4152ef09720a69e9cc018cf2d0..9377f181ab572f3dfbec149ecb772ebfef902d42 100644 (file)
--- a/src/msw/thread.cpp
+++ b/src/msw/thread.cpp
@@ -21,13 +21,15 @@
     #pragma hdrstop
 #endif
 
+#if wxUSE_THREADS
+
+#include "wx/thread.h"
+
 #ifndef WX_PRECOMP
     #include "wx/intl.h"
     #include "wx/app.h"
 #endif
 
-#if wxUSE_THREADS
-
 #include "wx/apptrait.h"
 #include "wx/scopeguard.h"
 
@@ -37,7 +39,6 @@
 
 #include "wx/except.h"
 #include "wx/module.h"
-#include "wx/thread.h"
 
 // must have this symbol defined to get _beginthread/_endthread declarations
 #ifndef _MT
@@ -436,6 +437,10 @@ public:
     // really start the thread (if it's not already dead)
     static THREAD_RETVAL DoThreadStart(wxThread *thread);
 
+    // call OnExit() on the thread
+    static void DoThreadOnExit(wxThread *thread);
+
+
     void KeepAlive()
     {
         if ( m_thread->IsDetached() )
@@ -477,14 +482,25 @@ private:
     wxThreadInternal& m_thrImpl;
 };
 
+/* static */
+void wxThreadInternal::DoThreadOnExit(wxThread *thread)
+{
+    wxTRY
+    {
+        thread->OnExit();
+    }
+    wxCATCH_ALL( wxTheApp->OnUnhandledException(); )
+}
+
+/* static */
 THREAD_RETVAL wxThreadInternal::DoThreadStart(wxThread *thread)
 {
+    wxON_BLOCK_EXIT1(DoThreadOnExit, thread);
+
     THREAD_RETVAL rc = (THREAD_RETVAL)-1;
 
     wxTRY
     {
-        wxON_BLOCK_EXIT_OBJ0(*thread, wxThread::OnExit);
-
         // store the thread object in the TLS
         if ( !::TlsSetValue(gs_tlsThisThread, thread) )
         {
@@ -500,6 +516,7 @@ THREAD_RETVAL wxThreadInternal::DoThreadStart(wxThread *thread)
     return rc;
 }
 
+/* static */
 THREAD_RETVAL THREAD_CALLCONV wxThreadInternal::WinThreadStart(void *param)
 {
     THREAD_RETVAL rc = (THREAD_RETVAL)-1;
@@ -513,14 +530,16 @@ THREAD_RETVAL THREAD_CALLCONV wxThreadInternal::WinThreadStart(void *param)
     // start the user code at all then
     const bool hasExited = thread->m_internal->GetState() == STATE_EXITED;
 
-    if ( !hasExited )
+    // run the thread function itself inside a SEH try/except block
+    wxSEH_TRY
     {
-        wxSEH_TRY
-        {
+        if ( hasExited )
+            DoThreadOnExit(thread);
+        else
             rc = DoThreadStart(thread);
-        }
-        wxSEH_HANDLE((THREAD_RETVAL)-1)
     }
+    wxSEH_HANDLE((THREAD_RETVAL)-1)
+
 
     // save IsDetached because thread object can be deleted by joinable
     // threads after state is changed to STATE_EXITED.
@@ -1363,4 +1382,3 @@ bool WXDLLIMPEXP_BASE wxIsWaitingForThread()
 #include "wx/thrimpl.cpp"
 
 #endif // wxUSE_THREADS
-