On Thu, 26 Sep 2024 17:06:38 GMT, Daniel Fuchs <dfu...@openjdk.org> wrote:
> `test/jdk/sun/net/www/ParseUtil_6380332.java`: shouldn't we actually keep > this test? I see that the code in ParseUtil which is tested here is still > there, and I don't think we'd want to touch ParseUtil given potential > compatibility issues that might arise. Hi Daniel, You're right, I looked at that test too fast. While there are similar tests testing `ParseUtil.toURI`, _ParseUtil_6380332.java_ tests the specific -1 port number case. Restored the test, and instead updated the summary in https://github.com/openjdk/jdk/pull/21096/commits/c90218e907b017b547f76a7010b9b87ca43f705a. Thanks. ------------- PR Comment: https://git.openjdk.org/jdk/pull/21096#issuecomment-2377519268