Re: RFR: JDK-8058932 - java/net/InetAddress/IPv4Formats.java failed because hello.foo.bar does exist

2014-10-01 Thread Dmitry Samersoff
Looks good for me! -Dmitry On 2014-10-01 02:26, Mark Sheppard wrote: > Thanks Tom and Dmitry > > last up best dressed ... > > .invalid as the test domain is a good recommendation > > change is now > > --- a/test/java/net/InetAddress/IPv4Formats.javaTue Sep 30 > 13:25:04 2014 +0100 >

RFR 8059450: Not quite correct code sample in javadoc

2014-10-01 Thread Ivan Gerasimov
Hello! This is a javadoc bug. In the code sample a redundant argument to a constructor of URI is passed. Probably a copy-paste error. BUGURL: https://bugs.openjdk.java.net/browse/JDK-8059450 WEBREV: http://cr.openjdk.java.net/~igerasim/8059450/0/webrev/ Sincerely yours, Ivan

Re: RFR: JDK-8058932 - java/net/InetAddress/IPv4Formats.java failed because hello.foo.bar does exist

2014-10-01 Thread Chris Hegarty
Reviewed. -Chris. On 1 Oct 2014, at 00:01, Dmitry Samersoff wrote: > Looks good for me! > > -Dmitry > > On 2014-10-01 02:26, Mark Sheppard wrote: >> Thanks Tom and Dmitry >> >> last up best dressed ... >> >> .invalid as the test domain is a good recommendation >> >> change is now >> >> -

Re: RFR: JDK-8058932 - java/net/InetAddress/IPv4Formats.java failed because hello.foo.bar does exist

2014-10-01 Thread Mark Sheppard
thanks Chris On 01/10/2014 15:58, Chris Hegarty wrote: Reviewed. -Chris. On 1 Oct 2014, at 00:01, Dmitry Samersoff wrote: Looks good for me! -Dmitry On 2014-10-01 02:26, Mark Sheppard wrote: Thanks Tom and Dmitry last up best dressed ... .invalid as the test domain is a good recommend

Re: RFR 8059450: Not quite correct code sample in javadoc

2014-10-01 Thread Chris Hegarty
On 1 Oct 2014, at 01:48, Ivan Gerasimov wrote: > Hello! > > This is a javadoc bug. > In the code sample a redundant argument to a constructor of URI is passed. > Probably a copy-paste error. > > BUGURL: https://bugs.openjdk.java.net/browse/JDK-8059450 > WEBREV: http://cr.openjdk.java.net/~igera

Re: RFR 8059450: Not quite correct code sample in javadoc

2014-10-01 Thread Ivan Gerasimov
Thanks Chris! On 01.10.2014 20:49, Chris Hegarty wrote: On 1 Oct 2014, at 01:48, Ivan Gerasimov wrote: Hello! This is a javadoc bug. In the code sample a redundant argument to a constructor of URI is passed. Probably a copy-paste error. BUGURL: https://bugs.openjdk.java.net/browse/JDK-80594