Dan Kenigsberg <dan...@redhat.com> added the comment:

Oh dear.

Thanks, Enzio, for pointing out that former patch is wrong. It is also quite 
naive, since the whole NATURE of toprettyprint() is to add whitespace to Text 
nodes. Tomas Lee's 
http://bugs.python.org/file11832/minidom-toprettyxml-01.patch made an effort to 
touch only "simple" Text nodes, that are confined within a single <elem></elem>.

I did not expect 
http://bugs.python.org/file23286/minidom-Text-toprettyxml.patch to get in so 
quickly, after the former one spent several years on queue. However now is time 
to fix it, possible by my second patch.

----------
Added file: http://bugs.python.org/file23294/minidom-Text-toprettyxml-02.patch

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

Reply via email to