RFR (JAXP) 8169450: StAX parse error if there is a newline in xml declaration
huizhe wang
huizhe.wang at oracle.com
Tue Feb 14 17:19:14 UTC 2017
Thanks Christoph. Will fix those comments.
-Joe
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
> Christoph
>
>
>> -----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
>> 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
>>
More information about the core-libs-dev
mailing list