Mark Dickinson <dicki...@gmail.com> added the comment:

Thanks for the patch!

I agree that the 'versionchanged' reference should be added in the docs.
 I also think that the test should be updated to check the exception type.

Here's a modified version of your patch that adds a test for
OverflowError to the capi tests, adds 'versionchanged', and rewords the
docs slightly.  I also took the liberty of rewording the docs for
PyLong_AsLongLong, to match.  I've tested this on a 64-bit linux
machine, and checked that the docs build properly.  I'll test on 32-bit
and 64-bit OS X later today.

Lisandro, does this updated patch work for you?

Added file: http://bugs.python.org/file13012/negative-to-unsigned2.diff

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue5175>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to