Raymond Hettinger <raymond.hettin...@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 <rep...@bugs.python.org>
<https://bugs.python.org/issue28988>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to