Martin Panter added the comment:

If this patch goes ahead, I think the ABC documentation should clarify which 
methods are checked for None and which aren’t. The datamodel.rst file will 
suggest None for any method, but ABC will only support it for Iterable and 
Hashable (I think).

Also, what is the point of the odd __getitem__() method in test_enumerate.py? 
Maybe you should use assertRaisesRegex() to check that the intended TypeError 
is actually raised.

----------
nosy: +martin.panter
stage: needs patch -> patch review

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

Reply via email to