On Wed, 17 Apr 2024 14:16:30 GMT, Daniel Fuchs <dfu...@openjdk.org> wrote:

> > Should we be setting any expectations by specifying what 
> > InetAddress.getHostAddress() will return for an Inet4Address constructed 
> > using this new Inet4Address.ofPosixLiteral() method? In its current form I 
> > believe it will continue to return a decimal representation of the IP 
> > address. My guess is that we want it to continue behaving that way?
> 
> In the InetAddress class level API documentation we have:
> 
> ```
>  * <p> For methods that return a textual representation as output
>  * value, the first form, i.e. a dotted-quad string, is used.
> ```
> 
> Do you think it should be reiterated in the `@apiNote` of the 
> `ofPosixLiteral` method?

The changes proposed in this PR introduce a new paragraph in the class level 
documentation just before the line which states the dotted-quad string. So I 
think it may not be clear enough whether dotted-quad string implies decimal 
values (`127.0.0.1`) or octal dotted-quad string (`0177.0.0.1`) or hexadecimal 
dotted-quad string (`0x7F.0.0.1`). So I think updating that sentence in class 
level documentation might help.

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

PR Comment: https://git.openjdk.org/jdk/pull/18493#issuecomment-2061393325

Reply via email to