On Tue, 10 May 2022 13:36:18 GMT, Daniel Fuchs <dfu...@openjdk.org> wrote:

>> Jaikiran Pai has updated the pull request incrementally with two additional 
>> commits since the last revision:
>> 
>>  - Daniel's review suggestion - add a test to verify the behaviour of the 
>> localAddress() default method implementation on HttpClient.Builder
>>  - Daniel's review suggestion - remove reference to "Internet Protocol" in 
>> javadoc
>
> test/jdk/java/net/httpclient/HttpClientBuilderTest.java line 283:
> 
>> 281:      * Tests that the default method implementation of
>> 282:      * {@link 
>> HttpClient,java.net.http.HttpClient.Builder#localAddress(InetAddress)} throws
>> 283:      * an {@link UnsupportedOperationException}
> 
> Same remark here

Otherwise things look good to me - you should update the CSR if it needs to be 
updated.

-------------

PR: https://git.openjdk.java.net/jdk/pull/6690

Reply via email to