RFR JDK-8027308: HttpURLConnection should better handle URLs with literal IPv6 addresses

[Pavel Rappo]

Alan, it looks simpler, indeed:

http://cr.openjdk.java.net/~prappo/8027308/webrev.01/src/share/classes/sun/net/www/protocol/http/HttpURLConnection.java.sdiff.html

I have removed the StringBuilder completely. It doesn't seem to be a performance critical part of the code :) But from the readability perspective this is a lot better.

-Pavel

On 9 Jun 2014, at 07:49, Alan Bateman <Alan.Bateman at oracle.com> wrote:

> On 09/06/2014 06:40, Chris Hegarty wrote:
>> This looks good to me Pavel.
>> 
>> -Chris.
> It looks okay to me too, I was just wondering if it would be simpler to use host.substring + sb.append(']').
> 
> -Alan
>