Raymond Hettinger <[email protected]> added the comment:
#632 should be left as is. Technically, the second "field++" is a dead store.
However, it is harmless and has some advantages. It keeps the the assignments
parallel and it reduces the chance of introducing a new bug if a new field is
added (i.e. like one of the reasons we use trailing commas on multiline dicts
and lists).
PyStructSequence_SET_ITEM(int_info, field++,
PyLong_FromLong(PyLong_SHIFT));
PyStructSequence_SET_ITEM(int_info, field++,
PyLong_FromLong(sizeof(digit)));
----------
_______________________________________
Python tracker <[email protected]>
<https://bugs.python.org/issue46280>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe:
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
