Mark Smith <mark.sm...@practicalpoetry.co.uk> added the comment:

Terry, thanks for the feedback!

I have added a patch, replacing the previous one, which deals with your points 
1 and 3 in the following ways:

1. I have used (an edited form of) your synopsis - I've removed the details of 
exactly what is returned from the iterator, as I thought it made the sentence 
structure slightly awkward - hope you don't mind. I've moved the old synopsis 
to the heading, which is where I intended it to be anyway.

3. I added the newline - I don't think this is really a problem, but neither is 
the fix :)

Regarding point 2: I agree with you - I struggled with whether to include 
references to these classes, eventually settling for what you see. I suppose 
they could be re-used for converting SAX events to DOM nodes, so it could be 
useful to know they are there.

SAX2DOM isn't even used internally, so is technically part of the module's 
public interface, but with such brittle behaviour that I think it should really 
be removed. It automatically adds children to their parent when they are parsed 
-- but the children will only be parsed if the buffer is large enough. If it 
isn't then you can get incorrectly empty or even partial subtrees.

----------
Added file: http://bugs.python.org/file18194/7635_2.patch

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

Reply via email to