Can you add a testcase for this? On Mon, Jun 4, 2012 at 9:01 PM, richard.oudkerk <python-checkins@python.org> wrote:
http://hg.python.org/cpython/rev/3baeb5e13dd2 changeset: 77348:3baeb5e13dd2 user: Richard Oudkerk <shibturn@gmail.com> date: Mon Jun 04 18:59:07 2012 +0100 summary: Fix potential NameError in multiprocessing.Condition.wait()
files: Lib/multiprocessing/synchronize.py | 3 +-- 1 files changed, 1 insertions(+), 2 deletions(-)
diff --git a/Lib/multiprocessing/synchronize.py b/Lib/multiprocessing/synchronize.py --- a/Lib/multiprocessing/synchronize.py +++ b/Lib/multiprocessing/synchronize.py @@ -216,7 +216,7 @@
try: # wait for notification or timeout - ret = self._wait_semaphore.acquire(True, timeout) + return self._wait_semaphore.acquire(True, timeout) finally: # indicate that this thread has woken self._woken_count.release() @@ -224,7 +224,6 @@ # reacquire lock for i in range(count): self._lock.acquire() - return ret
def notify(self): assert self._lock._semlock._is_mine(), 'lock is not owned'
-- Repository URL: http://hg.python.org/cpython
_______________________________________________ Python-checkins mailing list Python-checkins@python.org http://mail.python.org/mailman/listinfo/python-checkins