attribute names are case sensitive in XML too!

[lb42]

See http://teic.github.io/TCW/anyXMLproposal.html for example: the attribute is called minOccurs, not minoccurs.

While you're there, you shouldn't be removing the start and end tags from embedded comments either. See the bit of code at the end : the text "this code will/may need revising to deal with multiple values for the attribute " should be wrapped by <!-- and --> as it is in the source. You might possibly make a case for suppressing it entirely I suppose, but suppressing just its tags is a touch perverse.