]> git.saurik.com Git - wxWidgets.git/commitdiff
Add wxCRITSEC_DEFAULT and NON_RECURSIVE styles, updated and restored wxThread and...
authorRobert Roebling <robert@roebling.de>
Fri, 5 Sep 2008 08:06:07 +0000 (08:06 +0000)
committerRobert Roebling <robert@roebling.de>
Fri, 5 Sep 2008 08:06:07 +0000 (08:06 +0000)
git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@55465 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775

include/wx/thread.h
interface/wx/thread.h
src/msw/thread.cpp
src/osx/carbon/thread.cpp

index 4eed53d1bf99e4a026943f248839a858c53ec655..82ac361270c35bd169e15d14c50cebeef42b04fd 100644 (file)
@@ -218,13 +218,22 @@ private:
     #define wxCRITSECT_INLINE
 #endif // MSW/!MSW
 
+enum wxCriticalSectionType
+{
+    // recursive critical section
+    wxCRITSEC_DEFAULT,
+
+    // non-recursive critical section
+    wxCRITSEC_NON_RECURSIVE
+};
+
 // you should consider wxCriticalSectionLocker whenever possible instead of
 // directly working with wxCriticalSection class - it is safer
 class WXDLLIMPEXP_BASE wxCriticalSection
 {
 public:
     // ctor & dtor
-    wxCRITSECT_INLINE wxCriticalSection();
+    wxCRITSECT_INLINE wxCriticalSection( wxCriticalSectionType critSecType = wxCRITSEC_DEFAULT );
     wxCRITSECT_INLINE ~wxCriticalSection();
     // enter the section (the same as locking a mutex)
     wxCRITSECT_INLINE void Enter();
@@ -266,7 +275,8 @@ private:
 
 #if wxCRITSECT_IS_MUTEX && !defined(__WXMAC__)
     // implement wxCriticalSection using mutexes
-    inline wxCriticalSection::wxCriticalSection() { }
+    inline wxCriticalSection::wxCriticalSection( wxCriticalSectionType critSecType ) 
+       : m_mutex( critSecType == wxCRITSEC_DEFAULT ? wxMUTEX_RECURSIVE : wxMUTEX_DEFAULT )  { }
     inline wxCriticalSection::~wxCriticalSection() { }
 
     inline void wxCriticalSection::Enter() { (void)m_mutex.Lock(); }
index 3dd874afa0852f6456d612eecd262e046a7181ea..c57540f4b33f47f4bf020d59053bf70c7727b5d7 100644 (file)
@@ -250,24 +250,33 @@ public:
     */
 };
 
+/**
+   Possible critical section types
+*/
 
+enum wxCriticalSectionType
+{
+    wxCRITSEC_DEFAULT,
+      /** Recursive critical section under both Windows and Unix */
+
+    wxCRITSEC_NON_RECURSIVE  
+      /** Non-recursive critical section under Unix, recursive under Windows */
+};
 
 /**
     @class wxCriticalSection
 
     A critical section object is used for exactly the same purpose as
-    mutexes(). The only difference is that under Windows platform
+    a wxMutex. The only difference is that under Windows platform
     critical sections are only visible inside one process, while mutexes may be
     shared among processes, so using critical sections is slightly more
-    efficient. The terminology is also slightly different: mutex may be locked (or
-    acquired) and unlocked (or released) while critical section is entered and left
-    by the program.
+    efficient. The terminology is also slightly different: mutex may be locked
+    (or acquired) and unlocked (or released) while critical section is entered
+    and left by the program.
 
-    Finally, you should try to use
-    wxCriticalSectionLocker class whenever
+    Finally, you should try to use wxCriticalSectionLocker class whenever
     possible instead of directly using wxCriticalSection for the same reasons
-    wxMutexLocker is preferrable to
-    wxMutex - please see wxMutex for an example.
+    wxMutexLocker is preferrable to wxMutex - please see wxMutex for an example.
 
     @library{wxbase}
     @category{threading}
@@ -278,9 +287,10 @@ class wxCriticalSection
 {
 public:
     /**
-        Default constructor initializes critical section object.
+        Default constructor initializes critical section object. By default
+        critical sections are recursive under Unix and Windows.
     */
-    wxCriticalSection();
+    wxCriticalSection( wxCriticalSectionType critSecType = wxCRITSEC_DEFAULT );
 
     /**
         Destructor frees the resources.
@@ -301,6 +311,37 @@ public:
     void Leave();
 };
 
+/**
+  The possible thread kinds.
+*/
+enum wxThreadKind
+{
+    wxTHREAD_DETACHED,  /** Detached thread */
+    wxTHREAD_JOINABLE   /** Joinable thread */
+};
+
+/**
+  The possible thread errors.
+*/
+enum wxThreadError
+{
+    wxTHREAD_NO_ERROR = 0,      /** No error */
+    wxTHREAD_NO_RESOURCE,       /** No resource left to create a new thread */
+    wxTHREAD_RUNNING,           /** The thread is already running */
+    wxTHREAD_NOT_RUNNING,       /** The thread isn't running */
+    wxTHREAD_KILLED,            /** Thread we waited for had to be killed */
+    wxTHREAD_MISC_ERROR         /** Some other error */
+};
+
+/**
+   Defines the interval of priority
+*/
+enum
+{
+    WXTHREAD_MIN_PRIORITY      = 0u,
+    WXTHREAD_DEFAULT_PRIORITY  = 50u,
+    WXTHREAD_MAX_PRIORITY      = 100u
+};
 
 
 /**
@@ -801,6 +842,32 @@ public:
 };
 
 
+/**
+    The possible wxMutex kinds.
+*/
+enum wxMutexType
+{
+    wxMUTEX_DEFAULT,   /** Normal mutex: try to always use this one. Recursive under Windows. */
+
+    wxMUTEX_RECURSIVE  /** Recursive mutex: don't use these ones with wxCondition. */
+};
+
+
+/**
+    The possible wxMutex errors.
+*/
+enum wxMutexError
+{
+    wxMUTEX_NO_ERROR = 0,   /** operation completed successfully */
+    wxMUTEX_INVALID,        /** mutex hasn't been initialized */
+    wxMUTEX_DEAD_LOCK,      /** mutex is already locked by the calling thread */
+    wxMUTEX_BUSY,           /** mutex is already locked by another thread */
+    wxMUTEX_UNLOCKED,       /** attempt to unlock a mutex which is not locked */
+    wxMUTEX_TIMEOUT,        /** LockTimeout() has timed out */
+    wxMUTEX_MISC_ERROR      /** any other error */
+};
+
+
 
 /**
     @class wxMutex
@@ -816,14 +883,56 @@ public:
     thread is waiting) but using them is not recommended under Unix and they are
     @b not recursive there by default. The reason for this is that recursive
     mutexes are not supported by all Unix flavours and, worse, they cannot be used
-    with wxCondition. On the other hand, Win32 mutexes are
-    always recursive.
+    with wxCondition. On the other hand, Win32 mutexes are always recursive.
 
     For example, when several threads use the data stored in the linked list,
     modifications to the list should only be allowed to one thread at a time
     because during a new node addition the list integrity is temporarily broken
     (this is also called @e program invariant).
 
+    @code
+    // this variable has an "s_" prefix because it is static: seeing an "s_" in
+    // a multithreaded program is in general a good sign that you should use a
+    // mutex (or a critical section)
+    static wxMutex *s_mutexProtectingTheGlobalData;
+
+    // we store some numbers in this global array which is presumably used by
+    // several threads simultaneously
+    wxArrayInt s_data;
+
+    void MyThread::AddNewNode(int num)
+    {
+        // ensure that no other thread accesses the list
+        s_mutexProtectingTheGlobalList->Lock();
+
+        s_data.Add(num);
+
+        s_mutexProtectingTheGlobalList->Unlock();
+    }
+
+    // return true if the given number is greater than all array elements
+    bool MyThread::IsGreater(int num)
+    {
+        // before using the list we must acquire the mutex
+        wxMutexLocker lock(s_mutexProtectingTheGlobalData);
+
+        size_t count = s_data.Count();
+        for ( size_t n = 0; n < count; n++ )
+        {
+            if ( s_data[n] > num )
+                return false;
+        }
+
+        return true;
+    }
+    @endcode
+
+    Notice how wxMutexLocker was used in the second function to ensure that the
+    mutex is unlocked in any case: whether the function returns true or false
+    (because the destructor of the local object lock is always called). Using
+    this class instead of directly using wxMutex is, in general safer and is
+    even more so if your program uses C++ exceptions.
+
     @library{wxbase}
     @category{threading}
 
index 8dd5c820593035332de4b63f65e684e9e384a6fa..072df5b37329d145816eb91693eddd21cdc91342 100644 (file)
@@ -145,7 +145,7 @@ static bool gs_waitingForThread = false;
 // wxCriticalSection
 // ----------------------------------------------------------------------------
 
-wxCriticalSection::wxCriticalSection()
+wxCriticalSection::wxCriticalSection( wxCriticalSectionType WXUNUSED(critSecType) )
 {
     wxCOMPILE_TIME_ASSERT( sizeof(CRITICAL_SECTION) <= sizeof(wxCritSectBuffer),
                            wxCriticalSectionBufferTooSmall );
index 2b254f96cfb9ebfcbb6303e56dfb5aa8836e273e..f780d4bc03a3968331c78dd6254e62731041b0d9 100644 (file)
@@ -106,7 +106,7 @@ MPCriticalRegionID gs_guiCritical = kInvalidID;
 // wxCriticalSection
 // ----------------------------------------------------------------------------
 
-wxCriticalSection::wxCriticalSection()
+wxCriticalSection::wxCriticalSection( wxCriticalSectionType WXUNUSED(critSecType) )
 {
     MPCreateCriticalRegion( (MPCriticalRegionID*) &m_critRegion );
 }