RFR (JAXP) 8169450: StAX parse error if there is a newline in xml declaration
huizhe.wang at oracle.com
Tue Feb 14 17:19:14 UTC 2017
Thanks Christoph. Will fix those comments.
On 2/13/2017 11:56 PM, Langer, Christoph wrote:
> 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
>> -----Original Message-----
>> From: core-libs-dev [mailto:core-libs-dev-bounces at openjdk.java.net] On
>> Behalf Of huizhe wang
>> Sent: Dienstag, 14. Februar 2017 03:28
>> To: core-libs-dev at openjdk.java.net
>> Subject: RFR (JAXP) 8169450: StAX parse error if there is a newline in xml
>> 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.
More information about the core-libs-dev