[Cython] ct_DEF.__test__.large_nums doctest failing in 32bit

[Stefan Behnel]

Yaroslav Halchenko schrieb am 21.09.2015 um 16:26:
> was reported in Debian against 0.23.2 so tried blindly freshier snapshot
> from 0.23.x branch and that one is no go too:
> 
> ======================================================================
> FAIL: large_nums (line 95) (ct_DEF.__test__)
> Doctest: ct_DEF.__test__.large_nums (line 95)
> ----------------------------------------------------------------------
> Traceback (most recent call last):
>   File "/usr/lib/python2.7/doctest.py", line 2226, in runTest
>     raise self.failureException(self.format_failure(new.getvalue()))
> AssertionError: Failed doctest test for ct_DEF.__test__.large_nums (line 95)
>   File "/tmp/buildd/cython-0.23.2+git12-g2c9d175/build/work-dir/run/cpp/ct_DEF/ct_DEF.so", line unknown line number, in large_nums (line 95)
> 
> ----------------------------------------------------------------------
> File "/tmp/buildd/cython-0.23.2+git12-g2c9d175/build/work-dir/run/cpp/ct_DEF/ct_DEF.so", line ?, in ct_DEF.__test__.large_nums (line 95)
> Failed example:
>     print_large_number(n64)
> Expected:
>     -4294967295
> Got:
>     1

I cannot reproduce this locally (even tried a 32bit docker container), so I
blindly changed a couple of minor things in that branch. Could you give it
another try?

If that didn't help, here is a patch that adds a longness suffix to plain
integer literals based on their type (e.g. found after coercions). I'm not
sure yet if that's really a good idea, but would be helpful to know if it
fixes this test. Could you try both and report back?

Thanks!

Stefan

-------------- next part --------------
A non-text attachment was scrubbed...
Name: guess_longness.patch
Type: text/x-patch
Size: 1036 bytes
Desc: not available
URL: <http://mail.python.org/pipermail/cython-devel/attachments/20150922/e1103da3/attachment.bin>