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

Reply via email to