On Fri, 7 Oct 2022 07:52:53 GMT, Alan Bateman <al...@openjdk.org> wrote:
>> Daniel Fuchs has updated the pull request with a new target base due to a
>> merge or a rebase. The incremental webrev excludes the unrelated changes
>> brought in by the merge/rebase. The pull request contains three additional
>> commits since the last revision:
>>
>> - Merge branch 'master' into url-early-parsing-8293590
>> - Merge branch 'master' into url-early-parsing-8293590
>> - 8293590
>
> src/java.base/share/classes/java/net/URLStreamHandler.java line 515:
>
>> 513: if (host != null && isBuiltInHandler) {
>> 514: String errMsg = IPAddressUtil.checkHostString(host);
>> 515: if (errMsg != null) throw new
>> IllegalArgumentException(errMsg);
>
> Is it time for URLStreamHandler.setURL to document the
> IllegalArgumentException?
Good point. I have logged https://bugs.openjdk.org/browse/JDK-8294948 to
follow-up on this.
-------------
PR: https://git.openjdk.org/jdk/pull/10534
