RFR (JAXP) 8169450: StAX parse error if there is a newline in xml declaration
Lance Andersen
lance.andersen at oracle.com
Tue Feb 14 18:53:12 UTC 2017
Looks good… ship it.
> On Feb 14, 2017, at 1:41 PM, Daniel Fuchs <daniel.fuchs at oracle.com> wrote:
>
> On 14/02/17 17:21, huizhe wang wrote:
>> Thanks!
>>
>> Here's an updated webrev:
>> http://cr.openjdk.java.net/~joehw/jdk9/8169450/webrev/
>
> +1
>
> -- daniel
>
>>
>> -Joe
>>
>> On 2/14/2017 4:07 AM, Lance Andersen wrote:
>>> Looks good overall Joe. I would agree that I would clean up the minor
>>> comment alignment issues.
>>>
>>> Best
>>> Lance
>>>> On Feb 13, 2017, at 9:27 PM, huizhe wang <huizhe.wang at oracle.com
>>>> <mailto:huizhe.wang at oracle.com>> wrote:
>>>>
>>>> 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/
>>>> <http://cr.openjdk.java.net/%7Ejoehw/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
>>>>
>>>>
>>>
>>> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
>>> <http://oracle.com/us/design/oracle-email-sig-198324.gif><http://oracle.com/us/design/oracle-email-sig-198324.gif>
>>>
>>> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance
>>> Andersen| Principal Member of Technical Staff | +1.781.442.2037
>>> Oracle Java Engineering
>>> 1 Network Drive
>>> Burlington, MA 01803
>>> Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>
>>>
>>>
>>>
>>
>
<http://oracle.com/us/design/oracle-email-sig-198324.gif>
<http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
<http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>
More information about the core-libs-dev
mailing list