Amebis/wxWidgets
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix example showing wxMutex usage in documentation

Browse Source 
Fix wrong use of pointer and wrong variable name.

Closes #18778.
This commit is contained in:
Vadim Zeitlin
2020-06-01 15:04:45 +02:00
parent 68feb3e7ff
commit 192e8befb8
1 changed files with 7 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
interface/wx/thread.h
View File

@@ -1586,7 +1586,7 @@ enum wxMutexError
// this variable has an "s_" prefix because it is static: seeing an "s_" in // 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 // a multithreaded program is in general a good sign that you should use a
// mutex (or a critical section) // mutex (or a critical section)
static wxMutex *s_mutexProtectingTheGlobalData; static wxMutex s_mutexProtectingTheGlobalData;
// we store some numbers in this global array which is presumably used by // we store some numbers in this global array which is presumably used by
// several threads simultaneously // several threads simultaneously
@@ -1595,11 +1595,15 @@ enum wxMutexError
void MyThread::AddNewNode(int num) void MyThread::AddNewNode(int num)
{ {
// ensure that no other thread accesses the list // ensure that no other thread accesses the list
s_mutexProtectingTheGlobalList->Lock();
// Note that using Lock() and Unlock() explicitly is not recommended
// and only done here for illustrative purposes, prefer to use
// wxMutexLocker, as shown below, instead!
s_mutexProtectingTheGlobalData.Lock();
s_data.Add(num); s_data.Add(num);
s_mutexProtectingTheGlobalList->Unlock(); s_mutexProtectingTheGlobaData.Unlock();
} }
// return true if the given number is greater than all array elements // return true if the given number is greater than all array elements