Antoine Pitrou added the comment: > The problem is that you are duplicating an existing class (the > XMLParser) for no good reason
No. The logic was already in _IterParseIterator, just wrapped up differently. And the proof is that the original commit, while adding a feature, *removed* code from xml.etree: $ hg di -c f903cf864191 --stat Lib/xml/etree/ Lib/xml/etree/ElementTree.py | 209 +++++++++++++++++++++++++++++--------------------------------- 1 files changed, 97 insertions(+), 112 deletions(-) > This goes completely against the design of the existing API and > counters the reusability of the existing code. No it doesn't. It irritates you, for some reason, but your feelings are not a valid reason to revert a useful feature. And this issue is not for you to repeat the same stubborn line of argument again and again. I'm extremely bored, and I won't care about your comments anymore. ---------- _______________________________________ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue17741> _______________________________________ _______________________________________________ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
