On Thu, 10 Sep 2026 10:50:11 GMT, Daniel Fuchs <[email protected]> wrote:

>> Volkan Yazici has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Add `.psa()` case
>
> test/jdk/java/net/URI/Test.java line 1055:
> 
>> 1053:         test("http://[::FFFF:1.2.3.4"; + Long.MAX_VALUE + "]").x().z();
>> 1054:         test("http://1.2.3";).psa().x().z();
>> 1055:         test("http://1.2.3.300";).psa().x().z();
> 
> Can you add this additional test for plain IPv4-like literal?
> 
> Suggestion:
> 
>         test("http://1.2.3.300";).psa().x().z();
>         test("http://1.2.3.3"; + Long.MAX_VALUE).psa().x().z();

Implemented (along with some additional comments) in 61c9dfc384c.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/32786#discussion_r3987047035

Reply via email to