Nadeem Vawda <nadeem.va...@gmail.com> added the comment:

Attached is an updated patch with tests.

There don't seem to be any tests for PyLong_AsS[s]ize_t() and
PyLong_AsDouble(), so I added new ones for this issue. They should still
be expanded on at some point in the future, but for the meanwhile this
should be sufficient.

> I'm not sure whether it would be good to add information about the TypeError 
> to the docs.

Yeah, that doesn't seem necessary; raising TypeError in this sort of
situation is sufficiently typical behavior that explicitly documenting
it feels redundant.

----------
assignee:  -> nadeem.vawda
Added file: http://bugs.python.org/file23110/pylong-exceptions.diff

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

Reply via email to