Martin Panter added the comment:

What’s the point of the catch_warnings() and filterwarnings() calls in the 
tests? They don’t seem to be doing much; I think the CleanContext manager is 
already enabling warnings. Perhaps you could call simplefilter("error", 
ResourceWarning) instead.

Also I’m not enthusiastic about the iterparse() API when passing a file name. 
If it has to stay, maybe there should be an explicit way to clean it up without 
exhausting the iterator, like a generator.close() method.

----------
nosy: +martin.panter

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

Reply via email to