New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
XMLGenerator: nice <empty/> elements #45684
Comments
I modified xml.sax.saxutils.XMLGenerator, I used the version of saxutils.py that is shipped with python 2.5.1. |
patch for xml/sax/saxutils.py |
patch for _xmlplus.sax.saxutils.py |
Some (ugly) parsers insist on <container></ |
The attached patch makes this new feature optional, by passing the |
I have also added some unit tests to test the feature with the option |
Acting on a comment (with which I agree) that the "empty_elements_tag" |
This bug is tagged as easy and appears to already have a patch and a test posted from two years ago. What is still to be done here? |
I added the “needs review” keyword, to make this easy to find for reviewers, changed the version (new features don’t go in older versions; check if your patches still apply), and added MvL as nosy, since he’s listed as expert for the xml package in py3k/Misc/maintainers.rst |
I believe 3.2 is still open for new features. |
Updated version of the patch against current py3k svn (r83022) attached. This is a combined patch, including the previous xml.sax.saxutil changes (including the 'short_empty_elements' keyword) and the new tests. |
The patch looks clean to me, applied small edits to change it from git to svn. Changed test_sax got 7 errors on run. Changed saxutils 53 tests all passed. Can this please be committed. |
Can a committer look at this please. |
Patch looks good to me. |
So what still needs to happen to get this in? Patch still applies to current python 3.2 trunk (r85564). |
Committed Neil's patch (after adding docs) in r85671. Thanks. |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: