From ff931573ce9b0784366d06106dee69f61ff4490c Mon Sep 17 00:00:00 2001 From: Brett Cannon Date: Sun, 13 Jul 2008 01:20:19 +0000 Subject: [PATCH] Backport of 64903. --- Lib/dummy_thread.py | 9 +++------ Lib/test/test_dummy_thread.py | 1 + Misc/NEWS | 3 +++ 3 files changed, 7 insertions(+), 6 deletions(-) diff --git a/Lib/dummy_thread.py b/Lib/dummy_thread.py index a72c92722f..fac5720479 100644 --- a/Lib/dummy_thread.py +++ b/Lib/dummy_thread.py @@ -107,18 +107,15 @@ class LockType(object): aren't triggered and throw a little fit. """ - if waitflag is None: + if waitflag is None or waitflag: self.locked_status = True - return None - elif not waitflag: + return True + else: if not self.locked_status: self.locked_status = True return True else: return False - else: - self.locked_status = True - return True __enter__ = acquire diff --git a/Lib/test/test_dummy_thread.py b/Lib/test/test_dummy_thread.py index f274e0a0d0..58faeb4289 100644 --- a/Lib/test/test_dummy_thread.py +++ b/Lib/test/test_dummy_thread.py @@ -60,6 +60,7 @@ class LockTests(unittest.TestCase): #Make sure that an unconditional locking returns True. self.failUnless(self.lock.acquire(1) is True, "Unconditional locking did not return True.") + self.failUnless(self.lock.acquire() is True) def test_uncond_acquire_blocking(self): #Make sure that unconditional acquiring of a locked lock blocks. diff --git a/Misc/NEWS b/Misc/NEWS index c5cf068ea5..fa9c9a6b6c 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -59,6 +59,9 @@ Core and builtins Library ------- +- Issue #3339: dummy_thread.acquire() could return None which is not a valid + return value. + - Issue #3116 and #1792: Fix quadratic behavior in marshal.dumps(). - Issue #2682: ctypes callback functions no longer contain a cyclic -- 2.40.0