Message 294421 - Python tracker

➜

This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author	rhettinger
Recipients	espie, rhettinger
Date	2017-05-25.00:55:09
SpamBayes Score	-1.0
Marked as misclassified	Yes
Message-id	<1495673709.59.0.760799668428.issue30459@psf.upfronthosting.co.za>
In-reply-to

Content
The docs do make the claim of returning void: https://docs.python.org/3/c-api/list.html#c.PyList_SET_ITEM However, I think we should change the docs rather than changing the macro. This macro is very old and very widely used. Changing it is likely to break existing code which may rely on the current behavior. Also, I don't want to encourage code like what you saw in py-qt4. It is very indirect about what it is trying to express.

The docs do make the claim of returning void: https://docs.python.org/3/c-api/list.html#c.PyList_SET_ITEM

However, I think we should change the docs rather than changing the macro.  This macro is very old and very widely used.  Changing it is likely to break existing code which may rely on the current behavior.

Also, I don't want to encourage code like what you saw in py-qt4.  It is very indirect about what it is trying to express.

History
Date	User	Action	Args
2017-05-25 00:55:09	rhettinger	set	recipients: + rhettinger, espie
2017-05-25 00:55:09	rhettinger	set	messageid: <1495673709.59.0.760799668428.issue30459@psf.upfronthosting.co.za>
2017-05-25 00:55:09	rhettinger	link	issue30459 messages
2017-05-25 00:55:09	rhettinger	create