This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author tseaver
Recipients ajaksu2, ggenellina, jmg, rrwood, teajay_fr, tseaver, vdupras
Date 2010-05-02.20:16:45
SpamBayes Score 3.6065504e-08
Marked as misclassified No
Message-id <1272831407.77.0.838132918734.issue1777134@psf.upfronthosting.co.za>
In-reply-to
Content
The patch applies cleanly to the 2.6 branch, and with minimal fuzz to
the trunk.  Exsting tests pass in both cases, as does the new test.

The added testcase seems plainly correct.

The first hunk of the  diff to Lib/xml/dom/minidom.py is a little messy
/ hard to follow:

- It makes the module less PEP8 conformant (line length > 80, operator
  spacing)

- The 'onetextnode' flag is set, but never used.

The second hunk is somewhat problematic:  it changes 'Text.writexml' to
ignore any explicit 'indent' or 'newl' argument passed, which is a
backwards-incompatible behavior change.  Perhaps changing the default
values for those arguments to 'None', and then using the new code path
in that case, would be the better course, while falling back to the old
one if either is passed.

I'm not sure about the justification for the third hunk at all (removing
the newline at the end of the XML prologue).  There *can't* be any
meaningful whitespace outside of the root element, so we shouldn't have
round-trip issues if we leave it.
History
Date User Action Args
2010-05-02 20:16:48tseaversetrecipients: + tseaver, ggenellina, ajaksu2, jmg, teajay_fr, vdupras, rrwood
2010-05-02 20:16:47tseaversetmessageid: <1272831407.77.0.838132918734.issue1777134@psf.upfronthosting.co.za>
2010-05-02 20:16:45tseaverlinkissue1777134 messages
2010-05-02 20:16:45tseavercreate