Unicode overview added

[wxWidgets.git] / src / unix / threadpsx.cpp

diff --git a/src/unix/threadpsx.cpp b/src/unix/threadpsx.cpp
index aad7082be4d7ab0aa5f6342a5b2c28d388b977b8..513600f3b19133316f8de7f1c22eeab28ed4df22 100644 (file)
--- a/src/unix/threadpsx.cpp
+++ b/src/unix/threadpsx.cpp
@@ -101,6 +101,7 @@ public:
 wxMutex::wxMutex()
 {
     p_internal = new wxMutexInternal;
+    
     pthread_mutex_init( &(p_internal->p_mutex), (const pthread_mutexattr_t*) NULL );
     m_locked = 0;
 }
@@ -108,7 +109,7 @@ wxMutex::wxMutex()
 wxMutex::~wxMutex()
 {
     if (m_locked > 0)
-        wxLogDebug(_T("Freeing a locked mutex (%d locks)"), m_locked);
+        wxLogDebug(T("Freeing a locked mutex (%d locks)"), m_locked);
 
     pthread_mutex_destroy( &(p_internal->p_mutex) );
     delete p_internal;
@@ -119,7 +120,7 @@ wxMutexError wxMutex::Lock()
     int err = pthread_mutex_lock( &(p_internal->p_mutex) );
     if (err == EDEADLK)
     {
-        wxLogDebug(_T("Locking this mutex would lead to deadlock!"));
+        wxLogDebug(T("Locking this mutex would lead to deadlock!"));
 
         return wxMUTEX_DEAD_LOCK;
     }
@@ -155,7 +156,7 @@ wxMutexError wxMutex::Unlock()
     }
     else
     {
-        wxLogDebug(_T("Unlocking not locked mutex."));
+        wxLogDebug(T("Unlocking not locked mutex."));
 
         return wxMUTEX_UNLOCKED;
     }
@@ -319,7 +320,7 @@ void *wxThreadInternal::PthreadStart(void *ptr)
     // terminate the thread
     thread->Exit(status);
 
-    wxFAIL_MSG(_T("wxThread::Exit() can't return."));
+    wxFAIL_MSG(T("wxThread::Exit() can't return."));
 
     return NULL;
 }
@@ -388,7 +389,7 @@ wxThreadInternal::~wxThreadInternal()
 wxThreadError wxThreadInternal::Run()
 {
     wxCHECK_MSG( GetState() == STATE_NEW, wxTHREAD_RUNNING,
-                 _T("thread may only be started once after successful Create()") );
+                 T("thread may only be started once after successful Create()") );
 
     // the mutex was locked on Create(), so we will be able to lock it again
     // only when the thread really starts executing and enters the wait -
@@ -407,7 +408,7 @@ wxThreadError wxThreadInternal::Run()
 
 void wxThreadInternal::Wait()
 {
-    wxCHECK_RET( WasCancelled(), _T("thread should have been cancelled first") );
+    wxCHECK_RET( WasCancelled(), T("thread should have been cancelled first") );
 
     // if the thread we're waiting for is waiting for the GUI mutex, we will
     // deadlock so make sure we release it temporarily
@@ -451,7 +452,7 @@ void wxThreadInternal::Pause()
     // the state is set from the thread which pauses us first, this function
     // is called later so the state should have been already set
     wxCHECK_RET( m_state == STATE_PAUSED,
-                 _T("thread must first be paused with wxThread::Pause().") );
+                 T("thread must first be paused with wxThread::Pause().") );
 
     // don't pause the thread which is being terminated - this would lead to
     // deadlock if the thread is paused after Delete() had called Resume() but
@@ -466,7 +467,7 @@ void wxThreadInternal::Pause()
 void wxThreadInternal::Resume()
 {
     wxCHECK_RET( m_state == STATE_PAUSED,
-                 _T("can't resume thread which is not suspended.") );
+                 T("can't resume thread which is not suspended.") );
 
     // we will be able to lock this mutex only when Pause() starts waiting
     wxMutexLocker lock(m_mutexSuspend);
@@ -577,7 +578,7 @@ void wxThread::SetPriority(unsigned int prio)
 {
     wxCHECK_RET( ((int)WXTHREAD_MIN_PRIORITY <= (int)prio) &&
                  ((int)prio <= (int)WXTHREAD_MAX_PRIORITY),
-                 _T("invalid thread priority") );
+                 T("invalid thread priority") );
 
     wxCriticalSectionLocker lock(m_critsect);
 
@@ -606,7 +607,7 @@ void wxThread::SetPriority(unsigned int prio)
 
         case STATE_EXITED:
         default:
-            wxFAIL_MSG(_T("impossible to set thread priority in this state"));
+            wxFAIL_MSG(T("impossible to set thread priority in this state"));
     }
 }
 
@@ -632,7 +633,7 @@ wxThreadError wxThread::Pause()
 
     if ( p_internal->GetState() != STATE_RUNNING )
     {
-        wxLogDebug(_T("Can't pause thread which is not running."));
+        wxLogDebug(T("Can't pause thread which is not running."));
 
         return wxTHREAD_NOT_RUNNING;
     }
@@ -656,7 +657,7 @@ wxThreadError wxThread::Resume()
     }
     else
     {
-        wxLogDebug(_T("Attempt to resume a thread which is not paused."));
+        wxLogDebug(T("Attempt to resume a thread which is not paused."));
 
         return wxTHREAD_MISC_ERROR;
     }
@@ -668,6 +669,9 @@ wxThreadError wxThread::Resume()
 
 wxThread::ExitCode wxThread::Delete()
 {
+    if (IsPaused())
+      Resume();
+
     m_critsect.Enter();
     wxThreadState state = p_internal->GetState();
 
@@ -693,6 +697,8 @@ wxThread::ExitCode wxThread::Delete()
             // wait until the thread stops
             p_internal->Wait();
     }
+    //GL: As we must auto-destroy, the destruction must happen here.
+    delete this;
 
     return NULL;
 }
@@ -714,6 +720,8 @@ wxThreadError wxThread::Kill()
 
                 return wxTHREAD_MISC_ERROR;
             }
+           //GL: As we must auto-destroy, the destruction must happen here (2).
+           delete this;
 
             return wxTHREAD_NO_ERROR;
     }
@@ -761,7 +769,7 @@ wxThread::~wxThread()
     m_critsect.Enter();
     if (p_internal->GetState() != STATE_EXITED &&
         p_internal->GetState() != STATE_NEW)
-      wxLogDebug(_T("The thread is being destroyed althought it is still running ! The application may crash."));
+      wxLogDebug(T("The thread is being destroyed althought it is still running ! The application may crash."));
 
     m_critsect.Leave();
 
@@ -841,16 +849,18 @@ bool wxThreadModule::OnInit()
 
 void wxThreadModule::OnExit()
 {
-    wxASSERT_MSG( wxThread::IsMain(), _T("only main thread can be here") );
+    wxASSERT_MSG( wxThread::IsMain(), T("only main thread can be here") );
 
     // terminate any threads left
     size_t count = gs_allThreads.GetCount();
     if ( count != 0u )
-        wxLogDebug(_T("Some threads were not terminated by the application."));
+        wxLogDebug(T("Some threads were not terminated by the application."));
 
     for ( size_t n = 0u; n < count; n++ )
     {
-        gs_allThreads[n]->Delete();
+       // Delete calls the destructor which removes the current entry. We
+       // should only delete the first one each time.
+        gs_allThreads[0]->Delete();
     }
 
     // destroy GUI mutex