Re: RFR (JAXP) 8169450: StAX parse error if there is a newline in xml declaration

Thanks Lance, Daniel! -Joe On 2/14/2017 10:53 AM, Lance Andersen wrote: Looks good… ship it. On Feb 14, 2017, at 1:41 PM, Daniel Fuchs > wrote: On 14/02/17 17:21, huizhe wang wrote: Thanks! Here's an updated webrev: http://cr.openjdk.java.net/~joehw/jdk9/816

Re: RFR (JAXP) 8169450: StAX parse error if there is a newline in xml declaration

Looks good… ship it. > On Feb 14, 2017, at 1:41 PM, Daniel Fuchs 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 Anderse

Re: RFR (JAXP) 8169450: StAX parse error if there is a newline in xml declaration

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. B

Re: RFR (JAXP) 8169450: StAX parse error if there is a newline in xml declaration

Thanks! Here's an updated webrev: http://cr.openjdk.java.net/~joehw/jdk9/8169450/webrev/ -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

Re: RFR (JAXP) 8169450: StAX parse error if there is a newline in xml declaration

On 2/14/2017 3:16 AM, Daniel Fuchs wrote: Looks good Joe. I wonder about this though (which may be an issue for another time): 102 [25] Eq ::= S? '=' S? Do we support space (new line?) before and after the '=' sign? Yes, we do. I've added a couple of test cases to cover thi

Re: RFR (JAXP) 8169450: StAX parse error if there is a newline in xml declaration

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

Re: RFR (JAXP) 8169450: StAX parse error if there is a newline in xml declaration

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 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 addres

Re: RFR (JAXP) 8169450: StAX parse error if there is a newline in xml declaration

Looks good Joe. I wonder about this though (which may be an issue for another time): 102 [25] Eq ::= S? '=' S? Do we support space (new line?) before and after the '=' sign? best regards, -- daniel On 14/02/17 02:27, huizhe wang wrote: A quick fix for the error parsing xml d

RE: RFR (JAXP) 8169450: StAX parse error if there is a newline in xml declaration

zhe 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 th

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/ Checki