Comment 4 for bug 717254
Revision history for this message
| Louis Simard (louis-simard-deactivatedaccount) wrote | #4 |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
1b1ed1a
(Get the code!)
I'd advise against using str.isspace() and unicode.isspace() because SVG has a well-defined set of characters it considers whitespace. Though, in practice, even a form feed or vertical tab in <defs>, <metadata> or <g> isn't going to need being there.
The single unit test class looks sound, but checks for multiple things. It won't be obvious why the unit test fails, if it fails because of the whitespace-only <defs id="defs_two"> being kept. I'll add another unit test class and another file for the whitespace test.