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 ztane
Recipients martin.panter, serhiy.storchaka, xiang.zhang, ztane
Date 2016-07-10.09:49:48
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1468144188.53.0.306793344604.issue27473@psf.upfronthosting.co.za>
In-reply-to
Content
The previous code was perfectly fine with `-fwrapv` since it makes signed overflow behaviour defined. And afaik BDFLs stance is that signed integer overflow should be defined to wrap anyhow.

----

In my opinion the `-fwrapv` itself makes one proliferate all these insane wrap-checks; indeed I'd rather have them defined in a macro, something like

    if (PYSSIZE_OVERFLOWS_ON_ADD(va.len, vb.len)) {
        PyErr_NoMemory();
        goto done;
    }

    size = va.len + vb.len;

even though `-fwrapv` is defined; that way it'd be obvious what is supposed to happen there.
History
Date User Action Args
2016-07-10 09:49:48ztanesetrecipients: + ztane, martin.panter, serhiy.storchaka, xiang.zhang
2016-07-10 09:49:48ztanesetmessageid: <1468144188.53.0.306793344604.issue27473@psf.upfronthosting.co.za>
2016-07-10 09:49:48ztanelinkissue27473 messages
2016-07-10 09:49:48ztanecreate