-
Notifications
You must be signed in to change notification settings - Fork 10
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
Document limitation with comment in DTD #72
Conversation
@@ -192,9 +192,13 @@ private void finalizePreviousNode(XmlFilePosition endLocation) { | |||
private XMLStreamReader getXmlStreamReader() throws XMLStreamException { | |||
Reader reader = new StringReader(content); | |||
|
|||
XMLInputFactory factory = XMLInputFactory.newInstance(); | |||
// forcing the XMLInputFactory implementation class, in order to be sure that we are going to us the adequate |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use
DocumentType documentType = (DocumentType) document.getFirstChild(); | ||
assertRange(documentType, Location.NODE, 2, 0, 7, 2); | ||
Node lastChild = document.getLastChild(); | ||
// location of the node is wrong, should be line 8, event are not at the correct place |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
events
"<!ELEMENT menu (modulo)* >\n" + | ||
"]>\n" + | ||
"<menu value=\"foo\"></menu>").getDocument(); | ||
assertThat(document.getChildNodes().getLength()).isEqualTo(2); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
something is wrong with indentation
27461e6
to
b337b84
Compare
b337b84
to
c0adab7
Compare
Relates to https://jira.sonarsource.com/browse/SONARXML-73
Reported in Woodstox repository: FasterXML/woodstox#67