Re: XMLScanner-supplementary-characters

2014-09-11 Thread huizhe wang
On 9/11/2014 12:43 AM, Alan Bateman wrote: On 10/09/2014 23:11, huizhe wang wrote: Thanks Martin! Looks like the initial version of the jaxp port (from Xerces 2.7.1) missed the patch for reasons I don't know. I created a bug based on the very original bug report: https://bugs.openjdk.java.n

Re: XMLScanner-supplementary-characters

2014-09-11 Thread Alan Bateman
On 10/09/2014 23:11, huizhe wang wrote: Thanks Martin! Looks like the initial version of the jaxp port (from Xerces 2.7.1) missed the patch for reasons I don't know. I created a bug based on the very original bug report: https://bugs.openjdk.java.net/browse/JDK-8058175 Joe - what's the test c

Re: XMLScanner-supplementary-characters

2014-09-10 Thread Martin Buchholz
Thanks! Pushed. On Wed, Sep 10, 2014 at 4:14 PM, huizhe wang wrote: > > On 9/10/2014 3:20 PM, Martin Buchholz wrote: > > > > On Wed, Sep 10, 2014 at 3:11 PM, huizhe wang > wrote: > >> Thanks Martin! >> >> Looks like the initial version of the jaxp port (from Xerces 2.7.1) >> missed the patch f

Re: XMLScanner-supplementary-characters

2014-09-10 Thread huizhe wang
On 9/10/2014 3:20 PM, Martin Buchholz wrote: On Wed, Sep 10, 2014 at 3:11 PM, huizhe wang > wrote: Thanks Martin! Looks like the initial version of the jaxp port (from Xerces 2.7.1) missed the patch for reasons I don't know. I created a bug bas

Re: XMLScanner-supplementary-characters

2014-09-10 Thread Martin Buchholz
On Wed, Sep 10, 2014 at 3:11 PM, huizhe wang wrote: > Thanks Martin! > > Looks like the initial version of the jaxp port (from Xerces 2.7.1) missed > the patch for reasons I don't know. I created a bug based on the very > original bug report: > https://bugs.openjdk.java.net/browse/JDK-8058175 > >

Re: XMLScanner-supplementary-characters

2014-09-10 Thread huizhe wang
jaxp repo? Thanks, Joe On 9/10/2014 1:01 PM, Martin Buchholz wrote: Hi Huizhe, I have a strange contribution to share. Webrev: http://cr.openjdk.java.net/~martin/webrevs/openjdk9/XMLScanner-supplementary-characters/ <http://cr.openjdk.java.net/%7Emartin/webrevs/openjdk9/XMLScanner-supplementa

XMLScanner-supplementary-characters

2014-09-10 Thread Martin Buchholz
Hi Huizhe, I have a strange contribution to share. Webrev: http://cr.openjdk.java.net/~martin/webrevs/openjdk9/XMLScanner-supplementary-characters/ But I don't have a bug or a test case that I can share and I don't understand the code, except that it's a fix for supportin