Re: RFR [7u8]: 7166896: DocumentBuilder.parse(String uri) is not IPv6 enabled. It throws MalformedURLException

2012-07-12 Thread Paul Sandoz
Hi Joe,

On Jul 11, 2012, at 7:59 AM, Joe Wang wrote:

 Hi Paul,
 
 This is now for 7u8, so I started a new thread.
 
 As we discussed, I've removed the hack of using escapeNonUSAscii.  In this 
 regard, we're now in sync with the source code in Xerces.
 
 Here's the webrev:
 http://cr.openjdk.java.net/~joehw/7u8/7166896/webrev/
 

Looks OK to me. Note i am not an official reviewer, yet! so please don't commit 
cause i thought it looked OK ;-)

Paul.

Re: RFR [7u8]: 7166896: DocumentBuilder.parse(String uri) is not IPv6 enabled. It throws MalformedURLException

2012-07-12 Thread Lance Andersen - Oracle
+1

Best
Lance
On Jul 12, 2012, at 5:42 AM, Paul Sandoz wrote:

 Hi Joe,
 
 On Jul 11, 2012, at 7:59 AM, Joe Wang wrote:
 
 Hi Paul,
 
 This is now for 7u8, so I started a new thread.
 
 As we discussed, I've removed the hack of using escapeNonUSAscii.  In this 
 regard, we're now in sync with the source code in Xerces.
 
 Here's the webrev:
 http://cr.openjdk.java.net/~joehw/7u8/7166896/webrev/
 
 
 Looks OK to me. Note i am not an official reviewer, yet! so please don't 
 commit cause i thought it looked OK ;-)
 
 Paul.


Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering 
1 Network Drive 
Burlington, MA 01803
lance.ander...@oracle.com



Re: RFR [7u8]: 7166896: DocumentBuilder.parse(String uri) is not IPv6 enabled. It throws MalformedURLException

2012-07-12 Thread Joe Wang

Lance, Paul,

Thanks!

Joe

On 7/12/2012 3:55 AM, Lance Andersen - Oracle wrote:

+1

Best
Lance
On Jul 12, 2012, at 5:42 AM, Paul Sandoz wrote:


Hi Joe,

On Jul 11, 2012, at 7:59 AM, Joe Wang wrote:


Hi Paul,

This is now for 7u8, so I started a new thread.

As we discussed, I've removed the hack of using escapeNonUSAscii.  In this 
regard, we're now in sync with the source code in Xerces.

Here's the webrev:
http://cr.openjdk.java.net/~joehw/7u8/7166896/webrev/


Looks OK to me. Note i am not an official reviewer, yet! so please don't commit 
cause i thought it looked OK ;-)

Paul.


Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
lance.ander...@oracle.com