[issue28988] Switch dict and set structures to PyVarObject

Raymond Hettinger report at bugs.python.org
Thu Aug 22 05:17:24 EDT 2019


Raymond Hettinger <raymond.hettinger at gmail.com> added the comment:

> Since I don't see a benefit from this change I leave both patches
> to you Raymond.

After more reflection, I also don't see any point of doing this.  While there is a minor whiff of consistency when using Py_SIZE(), it makes the assignment look weird and inconsistent between filled and used.  I now favor local consistency and clarity (with in setobject.c) over trying to make it look exactly like some other modules.

----------
resolution:  -> rejected
stage: patch review -> resolved
status: open -> closed

_______________________________________
Python tracker <report at bugs.python.org>
<https://bugs.python.org/issue28988>
_______________________________________


More information about the Python-bugs-list mailing list