Martin Panter added the comment:

About testing that list(iterator) is empty after the iterator is closed, IMO 
this is an implementation detail. It would be equally (or more) sensible to 
raise an exception, like proposed for “async def” coroutines in Issue 25887. I 
suppose the main purpose of those tests is to ensure there is no leakage or 
warning, so maybe add some more filterwarnings() checks in those tests instead?

I left some other review comments.

----------

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

Reply via email to