Message409929
#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))); |
|
Date |
User |
Action |
Args |
2022-01-06 21:12:31 | rhettinger | set | recipients:
+ rhettinger, terry.reedy, eric.smith, serhiy.storchaka, pablogsal, nanjekyejoannah |
2022-01-06 21:12:31 | rhettinger | set | messageid: <1641503551.19.0.812845880517.issue46280@roundup.psfhosted.org> |
2022-01-06 21:12:31 | rhettinger | link | issue46280 messages |
2022-01-06 21:12:31 | rhettinger | create | |
|