Issue 13867: misleading comment in weakrefobject.h (original) (raw)

Issue13867

Created on 2012-01-25 23:01 by Jim.Jewett, last changed 2022-04-11 14:57 by admin. This issue is now closed.

Messages (5)
msg151983 - (view) Author: Jim Jewett (Jim.Jewett) * (Python triager) Date: 2012-01-25 23:01
http://hg.python.org/cpython/file/fec45282dc28/Include/weakrefobject.h#l54 The comment makes sense -- but doesn't appear to be true, so perhaps it is the macro that should change. /* This macro calls PyWeakref_CheckRef() last since that can involve a function call; this makes it more likely that the function call will be avoided. */ #define PyWeakref_Check(op) \ (PyWeakref_CheckRef(op) | PyWeakref_CheckProxy(op))
msg152142 - (view) Author: Antoine Pitrou (pitrou) * (Python committer) Date: 2012-01-27 23:39
Ha, good catch. The macro could be fixed, or the comment simply be dropped.
msg199229 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2013-10-08 18:26
New changeset 1800107873c0 by Georg Brandl in branch 'default': Closes #13867: remove untrue comment about PyWeakref_Check(). http://hg.python.org/cpython/rev/1800107873c0
msg199231 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2013-10-08 18:32
New changeset 1cd2fca12abf by Georg Brandl in branch '3.3': Closes #13867: remove untrue comment about PyWeakref_Check(). http://hg.python.org/cpython/rev/1cd2fca12abf
msg199232 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2013-10-08 18:33
New changeset 39e5ab118602 by Georg Brandl in branch '2.7': Closes #13867: remove untrue comment about PyWeakref_Check(). http://hg.python.org/cpython/rev/39e5ab118602
History
Date User Action Args
2022-04-11 14:57:26 admin set github: 58075
2013-10-08 18:33:01 python-dev set messages: +
2013-10-08 18:32:07 python-dev set messages: +
2013-10-08 18:26:55 python-dev set status: open -> closednosy: + python-devmessages: + resolution: fixedstage: resolved
2012-01-27 23:39:52 pitrou set messages: + versions: + Python 2.7, Python 3.2, Python 3.3
2012-01-27 23:36:27 terry.reedy set nosy: + fdrake, pitroucomponents: + Library (Lib), - Documentation, Extension Modules
2012-01-25 23:01:15 Jim.Jewett create