[PATCH] JDK-8201545 Clarify the return value of InetAddress.getByName/getAllName for empty host value

Chris Hegarty chris.hegarty at oracle.com
Mon Apr 23 11:33:27 UTC 2018


Thanks Jaikiran,

I think your patch looks good. I filed the following CSR to track
the Java SE API ( javadoc ) change.

https://bugs.openjdk.java.net/browse/JDK-8202139

Once approved, I can sponsor this for you.

-Chris.

On 20/04/18 11:08, Jaikiran Pai wrote:
> Hi,
> 
> The attached patch addresses the issue noted in [1], by updating the 
> javadoc of InetAddress.getByName and InetAddress.getAllByName to clarify 
> that these methods return a loopback address, if the host parameter is 
> an empty string (same behaviour as host == null). The patch also updates 
> an existing test case to test these methods for both null and empty 
> parameter values.
> 
> After looking at existing tests for InetAddress, I felt the existing 
> GetLoopbackAddress.java test case is closest to what we are testing 
> here. One thing I need input on, for the GetLoopbackAddress.java test 
> class, is the value of @summary. Should I update it to include a summary 
> of this new test too, or should I remove it altogether? I have anyway 
> updated the @bug to include the JIRA id of this issue. I am open to 
> creating a fresh new test case class just for this issue, if that's better.
> 
> [1] https://bugs.openjdk.java.net/browse/JDK-8201545
> 
> -Jaikiran
> 


More information about the net-dev mailing list