search

sshyran / genxdm

Automatically exported from code.google.com/p/genxdm
0 stars 0 forks source link

XmlEventVisitor doesn't properly handle files without <?xml ?> decl #15

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
it appears that the XmlEventVisitor code did not handle the case properly where 
the standard <?xml version="1.0" ?> declaration does not appear.

It appears that this stems from XMLEventReader - it doesn't produce a "start 
document" event if the <?xml ... does not appear. Consequently, the 
XmlNodeBuilder doesn't create a document, and my test cases, instead of getting 
a document for a root node, get a PI.

Original issue reported on code.google.com by aale...@gmail.com on 12 Oct 2010 at 4:36

GoogleCodeExporter commented 8 years ago 
There's already code that tries to wrap a fragment in a document.  We need to 
revisit that code, and make it work for every case, unless someone 
*specifically* wants to parse a fragment.

Original comment by aale...@gmail.com on 12 Oct 2010 at 4:36

GoogleCodeExporter commented 8 years ago 
This issue was closed by revision r50.

Original comment by aale...@gmail.com on 21 Oct 2010 at 4:06