Hi Joe, this looks right to me, quite straightforward.
I just have some formatting nits for the test: 1. Line 107 (XMLInputFactory xif=): space missing before '=' 2. Method comments on test methods would look nicer if formatted like: /* * line 1 * line 2 * ... */ ... I want to say that the '*' is missing in the beginning of the middle lines 3. Several whitespace at the end of lines - but jcheck should find out Best regards Christoph > -----Original Message----- > From: core-libs-dev [mailto:core-libs-dev-boun...@openjdk.java.net] On > Behalf Of huizhe wang > Sent: Dienstag, 14. Februar 2017 03:28 > To: core-libs-dev@openjdk.java.net > Subject: RFR (JAXP) 8169450: StAX parse error if there is a newline in xml > declaration > > A quick fix for the error parsing xml declaration. This is one of the > three outstanding issues in the xml area that must be addressed for the > coming development deadline. > > JBS: https://bugs.openjdk.java.net/browse/JDK-8169450 > webrev: http://cr.openjdk.java.net/~joehw/jdk9/8169450/webrev/ > > Checking whether a space follows "<?xml" was sufficient for DOM/SAX for > which the base parser was originally created since the xml declaration > was normalized during version detection. StAX, sharing the base parser > after it was added to the JDK, however, does not use the same version > detection/normalization process and would incorrectly parse the > declaration if there are other whitespaces. > > Thanks, > Joe >