INADA Naoki added the comment:

patch is LGTM.

But there is one hidden inconsistency:

>>> r = range(2**100)
>>> type(iter(r))
<class 'longrange_iterator'>
>>> type(iter(r))(1, 1, 0)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: cannot create 'longrange_iterator' instances

Should we have same tp_new method for longrange_iterator?

----------
nosy: +inada.naoki

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

Reply via email to