Antoine Pitrou added the comment:

The implementation is a bit weird. Why take the lock a second time instead of 
simply reimplementing the release() method?
(it's a 5-liner)

By the way, Semaphore.acquire could probably use Condition.wait_for.

----------
nosy: +pitrou

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue19158>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to