Serhiy Storchaka added the comment:

> The patch looks OK, buy can you provide a way to reproduce the error (if you 
> get any)?

No, I have not any. I am even not sure tabnanny works at all. But this bug is 
obvious. So, let's fix it and go on.

> Should we add tests for tabnanny?

This will be good, but it is a different issue. I'm not ready to write a test.

> I tried to get an error from tabnanny but the only thing I got was a 
> ResourceWarning (that can be easily fixed by a finally: f.close() near the 
> end of the check() method).

I can't see a ResourceWarning. A "finally: f.close()" already exists near the 
end of the check() method.

----------

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

Reply via email to