On Thu, 12 Mar 2026 14:14:18 GMT, Daniel Fuchs <[email protected]> wrote:
>> Eirik Bjørsnøs has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Unify 'path' variables
>
> test/jdk/java/net/URL/Constructor.java line 267:
>
>> 265: new Entry(null, "file:/path#?", "file:/path#?"),
>> 266: new Entry("file:/path", "path?#", "file:/path?#"),
>> 267: new Entry(null, "file:", "file:"),
>
> Should we add some entries of the form `<protocol>:///...` ? This also
> results in an empty/null authority component.
Thanks, added cases as suggested. Test runs successfully against both PR and
mainline version of the method.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/30151#discussion_r2925071665
