changeset:   98550:3719e842a7b1
branch:      3.5
parent:      98545:28fcd7f13613
parent:      98549:6108d30dde21
user:        Benjamin Peterson <benjamin@python.org>
date:        Mon Oct 05 22:00:33 2015 -0700
files:       Lib/test/lock_tests.py Lib/threading.py Misc/ACKS Misc/NEWS
description:
merge 3.4 (#25319)


diff -r 28fcd7f13613 -r 3719e842a7b1 Lib/test/lock_tests.py
--- a/Lib/test/lock_tests.py	Mon Oct 05 16:23:13 2015 -0700
+++ b/Lib/test/lock_tests.py	Mon Oct 05 22:00:33 2015 -0700
@@ -394,6 +394,14 @@
         b.wait_for_finished()
         self.assertEqual(results, [True] * N)
 
+    def test_reset_internal_locks(self):
+        evt = self.eventtype()
+        old_lock = evt._cond._lock
+        evt._reset_internal_locks()
+        new_lock = evt._cond._lock
+        self.assertIsNot(new_lock, old_lock)
+        self.assertIs(type(new_lock), type(old_lock))
+
 
 class ConditionTests(BaseTestCase):
     """
diff -r 28fcd7f13613 -r 3719e842a7b1 Lib/threading.py
--- a/Lib/threading.py	Mon Oct 05 16:23:13 2015 -0700
+++ b/Lib/threading.py	Mon Oct 05 22:00:33 2015 -0700
@@ -499,7 +499,7 @@
 
     def _reset_internal_locks(self):
         # private!  called by Thread._reset_internal_locks by _after_fork()
-        self._cond.__init__()
+        self._cond.__init__(Lock())
 
     def is_set(self):
         """Return true if and only if the internal flag is true."""
diff -r 28fcd7f13613 -r 3719e842a7b1 Misc/ACKS
--- a/Misc/ACKS	Mon Oct 05 16:23:13 2015 -0700
+++ b/Misc/ACKS	Mon Oct 05 22:00:33 2015 -0700
@@ -1352,6 +1352,7 @@
 Rafal Smotrzyk
 Eric Snow
 Dirk Soede
+Nir Soffer
 Paul Sokolovsky
 Evgeny Sologubov
 Cody Somerville
diff -r 28fcd7f13613 -r 3719e842a7b1 Misc/NEWS
--- a/Misc/NEWS	Mon Oct 05 16:23:13 2015 -0700
+++ b/Misc/NEWS	Mon Oct 05 22:00:33 2015 -0700
@@ -150,6 +150,9 @@
 - Issue #24881: Fixed setting binary mode in Python implementation of FileIO
   on Windows and Cygwin.  Patch from Akira Li.
 
+- Issue #25319: When threading.Event is reinitialized, the underlying condition
+  should use a regular lock rather than a recursive lock.
+
 - Issue #21112: Fix regression in unittest.expectedFailure on subclasses.
   Patch from Berker Peksag.