[Python-Dev] please back out changeset f903cf864191 before alpha-2 (original) (raw)

Stephen J. Turnbull stephen at xemacs.org
Sun Aug 25 02:55:40 CEST 2013

Eli Bendersky writes:

I'm strongly opposed to reverting [the change to ElementTree] because it cleaned up messy code duplication and actually make the code size smaller. While I agree that the API of incremental parsing should be given another look, IncrementalParser can also be seen as an implementation detail of iterparse().

Except that its API is familiar and cleaner. Does any current application depend on not doing whatever it is that the new API does that IncrementalParser does do? If not, why not keep the API of IncrementalParser and shim the new code in under that?

Thus, it's probably OK to revert the documentation part of the commit to not mention IncrementalParser at all,

FWIW, as somebody who can recall using ET exactly one, IncrementalParser is what I used.

More information about the Python-Dev mailing list