RFR: 8267840: Improve URLStreamHandler.parseURL() [v2]

Сергей Цыпанов github.com+10835776+stsypanov at openjdk.java.net
Mon Jul 5 13:37:57 UTC 2021


On Fri, 2 Jul 2021 13:57:47 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:

>> Сергей Цыпанов 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 8267840
>>  - Merge branch 'master' into 8267840
>>  - 8267840: Improve URLStreamHandler.parseURL()
>
> src/java.base/share/classes/java/net/URLStreamHandler.java line 275:
> 
>> 273:                 path = "/";
>> 274:             } else {
>> 275:                 path = path.substring(0, ind).concat("/");
> 
> would that be equivalent to 
> 
> path = path.substring(0, ind + 1);
> 
> given that ind = path.lastIndexOf('/') ?

Right, fixed!

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

PR: https://git.openjdk.java.net/jdk/pull/4526


More information about the net-dev mailing list