Re: [jdk10] (XXS) RFR 8189631 : Missing space in the javadoc for InetAddress.createNameService()

2017-11-14 Thread Bradford Wetmore
+1. Brad On 11/14/2017 11:47 AM, Roger Riggs wrote: +1, Reviewed Roger On 11/14/2017 2:45 PM, Ivan Gerasimov wrote: Hello! A typo in the javadoc: --- a/src/java.base/share/classes/java/net/InetAddress.java +++ b/src/java.base/share/classes/java/net/InetAddress.java @@ -1133,7 +1133,7 @@

Re: [jdk10] (XXS) RFR 8189631 : Missing space in the javadoc for InetAddress.createNameService()

2017-11-14 Thread Roger Riggs
+1, Reviewed Roger On 11/14/2017 2:45 PM, Ivan Gerasimov wrote: Hello! A typo in the javadoc: --- a/src/java.base/share/classes/java/net/InetAddress.java +++ b/src/java.base/share/classes/java/net/InetAddress.java @@ -1133,7 +1133,7 @@ /**   * Create an instance of the NameService i

[jdk10] (XXS) RFR 8189631 : Missing space in the javadoc for InetAddress.createNameService()

2017-11-14 Thread Ivan Gerasimov
Hello! A typo in the javadoc: --- a/src/java.base/share/classes/java/net/InetAddress.java +++ b/src/java.base/share/classes/java/net/InetAddress.java @@ -1133,7 +1133,7 @@ /** * Create an instance of the NameService interface based on - * the setting of the {@codejdk.net.hosts.file} s

Re: RFR 8191172/10, Problem list java/net/httpclient/websocket/ConnectionHandover.java

2017-11-14 Thread Chris Hegarty
> On 14 Nov 2017, at 08:26, Felix Yang wrote: > > Hi all, > > please review a minor change to problem list > java/net/httpclient/websocket/ConnectionHandover.java temporarily. It has > been changed a lot in sandbox, so this test can be resumed after future > updates from sandbox. > > Bu

RFR 8191172/10, Problem list java/net/httpclient/websocket/ConnectionHandover.java

2017-11-14 Thread Felix Yang
Hi all,     please review a minor change to problem list java/net/httpclient/websocket/ConnectionHandover.java temporarily. It has been changed a lot in sandbox, so this test can be resumed after future updates from sandbox. Bug:   https://bugs.openjdk.java.net/browse/JDK-8191172 Origi