From: Jesse Noller Date: Tue, 31 Mar 2009 03:25:07 +0000 (+0000) Subject: merge 70783 to py3k X-Git-Tag: v3.1a2~147 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=f8d00855109d98d364882d46b2060a46af6456c3;p=python merge 70783 to py3k --- diff --git a/Lib/test/test_multiprocessing.py b/Lib/test/test_multiprocessing.py index ed8c62630b..29f823de08 100644 --- a/Lib/test/test_multiprocessing.py +++ b/Lib/test/test_multiprocessing.py @@ -548,6 +548,10 @@ class _TestLock(BaseTestCase): self.assertEqual(lock.release(), None) self.assertRaises((AssertionError, RuntimeError), lock.release) + def test_lock_context(self): + with self.Lock(): + pass + class _TestSemaphore(BaseTestCase): diff --git a/Misc/ACKS b/Misc/ACKS index 03ed92de10..bd300de67e 100644 --- a/Misc/ACKS +++ b/Misc/ACKS @@ -257,6 +257,7 @@ Dinu Gherman Jonathan Giddy Johannes Gijsbers Michael Gilfix +Tim Golden Chris Gonnerman David Goodger Hans de Graaff @@ -794,4 +795,3 @@ Siebren van der Zee Uwe Zessin Tarek ZiadŽ Peter Åstrand -Jesse Noller diff --git a/Misc/NEWS b/Misc/NEWS index de4245a9c6..894aa71888 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -49,6 +49,9 @@ Core and Builtins Library ------- +- Issue #5261: Patch multiprocessing's semaphore.c to support context + manager use: "with multiprocessing.Lock()" works now. + - Issue #5236: Change time.strptime() to only take strings. Didn't work with bytes already but the failure was non-obvious. diff --git a/Modules/_multiprocessing/semaphore.c b/Modules/_multiprocessing/semaphore.c index 9c8c445eb6..40bd7c3c88 100644 --- a/Modules/_multiprocessing/semaphore.c +++ b/Modules/_multiprocessing/semaphore.c @@ -546,7 +546,7 @@ static PyMethodDef semlock_methods[] = { "acquire the semaphore/lock"}, {"release", (PyCFunction)semlock_release, METH_NOARGS, "release the semaphore/lock"}, - {"__enter__", (PyCFunction)semlock_acquire, METH_VARARGS, + {"__enter__", (PyCFunction)semlock_acquire, METH_VARARGS | METH_KEYWORDS, "enter the semaphore/lock"}, {"__exit__", (PyCFunction)semlock_release, METH_VARARGS, "exit the semaphore/lock"},