diff options
author | Kristj?n Valur J?nsson <sweskman@gmail.com> | 2013-03-19 20:18:37 -0700 |
---|---|---|
committer | Kristj?n Valur J?nsson <sweskman@gmail.com> | 2013-03-19 20:18:37 -0700 |
commit | 1010538749db6a0f9fc5143c8bd78528ba6c2b0d (patch) | |
tree | 0547fb2b0e4d82204ae6dbf954886d2fe965c370 /Python | |
parent | 84d910b372375ad37c9dcd6fb34f3102d4aad4bb (diff) | |
download | cpython-1010538749db6a0f9fc5143c8bd78528ba6c2b0d.tar.gz |
Issue #15038 : Fixing the condition broadcast and docs.
Diffstat (limited to 'Python')
-rw-r--r-- | Python/condvar.h | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/Python/condvar.h b/Python/condvar.h index 72b08f98c0..e022dc7938 100644 --- a/Python/condvar.h +++ b/Python/condvar.h @@ -163,10 +163,9 @@ PyCOND_TIMEDWAIT(PyCOND_T *cond, PyMUTEX_T *mut, long us) Generic emulations of the pthread_cond_* API using earlier Win32 functions can be found on the Web. - The following read can be edificating (or not): + The following read can be give background information to these issues, + but the implementations are all broken in some way. http://www.cse.wustl.edu/~schmidt/win32-cv-1.html - - See also */ typedef CRITICAL_SECTION PyMUTEX_T; @@ -297,9 +296,10 @@ PyCOND_SIGNAL(PyCOND_T *cv) Py_LOCAL_INLINE(int) PyCOND_BROADCAST(PyCOND_T *cv) { - if (cv->waiting > 0) { - return ReleaseSemaphore(cv->sem, cv->waiting, NULL) ? 0 : -1; - cv->waiting = 0; + int waiting = cv->waiting; + if (waiting > 0) { + cv->waiting = 0; + return ReleaseSemaphore(cv->sem, waiting, NULL) ? 0 : -1; } return 0; } |