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 gregory.p.smith
Recipients gregory.p.smith, gvanrossum
Date 2007-10-16.00:53:37
SpamBayes Score 0.019517621
Marked as misclassified No
Message-id <1192496019.83.0.342182192449.issue1261@psf.upfronthosting.co.za>
In-reply-to
Content
Ok, I believe I've fixed everything you commented on, accidentally
included in initials used as a search placeholder included. :)

> +                /* XXX: this depends on a signed integer overflow to
< >0 */
> +                /* C compilers, including gcc, do -NOT- guarantee
>this. */

> (And repeated twice more.)  Wouldn't it be better to write this in a way
> that doesn't require this XXX comment?  ISTR that we already had one
> area where we had to fight with gcc because it had proved to itself that
> something could never be negative, even though it could be due to
> overflow.  The GCC authors won. :-(

This code was copied as is from stringobject.c, I just added the XXX
notes upon reading it.  There is an existing unittest to try and make
sure the code has compiled properly with these tests (its not
comprehensive, it'll only test one of these three test cases). 
Regardless, yes, it needs fixing.  I didn't want to take that on that as
part of this patch.
Files
File name Uploaded
bytes-pep3137-methods-04.diff.txt gregory.p.smith, 2007-10-16.00:53:37
History
Date User Action Args
2007-10-16 00:53:40gregory.p.smithsetspambayes_score: 0.0195176 -> 0.019517621
recipients: + gregory.p.smith, gvanrossum
2007-10-16 00:53:39gregory.p.smithsetspambayes_score: 0.0195176 -> 0.0195176
messageid: <1192496019.83.0.342182192449.issue1261@psf.upfronthosting.co.za>
2007-10-16 00:53:39gregory.p.smithlinkissue1261 messages
2007-10-16 00:53:38gregory.p.smithcreate