STINNER Victor victor.stin...@haypocalc.com added the comment:
Is the assert still needed?
The assertion is in RLock()._acquire_restore(), not in RLock._release_save(). I
prefer to keep it, it doesn't hurt.
close this issue because I commited my fix.
--
resolution: - fixed
status:
Roundup Robot devnull@devnull added the comment:
New changeset 996b9c9dc10a by Victor Stinner in branch 'default':
Issue #11005, issue #11915: fix issue number of commit 2c0da1c4f063.
http://hg.python.org/cpython/rev/996b9c9dc10a
--
nosy: +python-dev
STINNER Victor victor.stin...@haypocalc.com added the comment:
The commit:
New changeset 2c0da1c4f063 by Victor Stinner in branch 'default':
Issue #11915: threading.RLock()._release_save() raises a RuntimeError if the
http://hg.python.org/cpython/rev/2c0da1c4f063
--
Antoine Pitrou pit...@free.fr added the comment:
Here is a patch: Antoine, would you like to review it?
--
keywords: +patch
nosy: +pitrou
Added file: http://bugs.python.org/file21636/rlock_release_save.patch
Well, it looks ok to me. Is the assert still needed?
--
STINNER Victor victor.stin...@haypocalc.com added the comment:
Here is a patch: Antoine, would you like to review it?
--
keywords: +patch
nosy: +pitrou
Added file: http://bugs.python.org/file21636/rlock_release_save.patch
___
Python tracker
New submission from STINNER Victor victor.stin...@haypocalc.com:
Attached script (bug2.py) ends with:
$ ~/prog/SVN/py3k/python bug2.py
python: ./Modules/_threadmodule.c:399:
