[issue19047] Assorted weakref docs improvements

2013-09-22 Thread Richard Oudkerk
Richard Oudkerk added the comment: Thanks for the doc cleanup -- I am rather busy right now. Note that stuff does still get replaced by None at shutdown, and this can still produce errors, even if they are much harder to trigger. If I run the following program import _weakref import

[issue19047] Assorted weakref docs improvements

2013-09-22 Thread Roundup Robot
Roundup Robot added the comment: New changeset caa16423b324 by Nick Coghlan in branch 'default': Close #19047: weakref doc cleanups http://hg.python.org/cpython/rev/caa16423b324 -- nosy: +python-dev resolution: -> fixed stage: needs patch -> committed/rejected status: open -> closed __

[issue19047] Assorted weakref docs improvements

2013-09-19 Thread Nick Coghlan
Nick Coghlan added the comment: I was initially thinking to myself "I have a source checkout right here, I should just fix it, even though I'm at work and want to go home"... and then I realised the potential scope of the fixes needed, given the longstanding misbehaviours these docs assume still

[issue19047] Assorted weakref docs improvements

2013-09-19 Thread Antoine Pitrou
Antoine Pitrou added the comment: Doc patches certainly welcome :-) -- nosy: +pitrou, sbt ___ Python tracker ___ ___ Python-bugs-list

[issue19047] Assorted weakref docs improvements

2013-09-19 Thread Nick Coghlan
Nick Coghlan added the comment: Changing the title, since I spotted a few other problems as well. The weakref docs still refer to module globals being set to None during shutdown. That is no longer the case. There are also some assumptions about cycles with __del__ methods not being garbage co