[issue1706039] Added clearerr() to clear EOF state

Scott Dial report at bugs.python.org
Wed Mar 9 08:32:22 CET 2011


Scott Dial <scott at scottdial.com> added the comment:

I've updated the patch to apply to the current tip. (This patch was an opportunity for me to update to an Hg workflow.)

Alexander, I disagree with you about the tests. The unittests use the exact same pattern/model that testIteration uses. I find your complaint that a unittest, which does test the feature under question, is not good enough, despite the prevailing unittests being designed in the same manner, to be absurd. Practicality beats purity -- a test that works is better than no test at all.

By rejecting unittests on the merits of its coding style, you are creating a double-standard for people like me (outside of the core committers), which eventually wears out my interest in helping you get this improvement into your project. I have been chased around this obstacle course before. For example, issue5949, when I was asked to provide unittests for a module that had *none*, for a 3-line patch that nobody disagreed it being correct. I had a vested interest in jumping through the obstacles of getting that patch in, but here I am again being blocked from making a 3-line patch, except this time, purely for stylistic reasons.

----------
Added file: http://bugs.python.org/file21060/issue1706039-py3k.patch

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue1706039>
_______________________________________


More information about the Python-bugs-list mailing list