RFR: Get resource path from URI to avoid getting spurious leading path separator on Windows

Erik Duveblad via github.com duke at openjdk.java.net
Mon Aug 5 13:16:04 UTC 2019


On Mon, 5 Aug 2019 11:00:08 GMT, JornVernee via github.com <duke at openjdk.java.net> wrote:

> On Fri, 2 Aug 2019 15:03:49 GMT, JornVernee via github.com <duke at openjdk.java.net> wrote:
> 
>> One more little pathing problem I ran into when running GitWebrev through a debugger.
>> 
>> The old code uses `URL::getPath`, but this results in a leading slash before the path, and on Windows absolute paths contain drive letters, so you get something like `/C:/a/b/c`, which is not a valid path syntax.
>> 
>> We can first convert the URL to a URI, for which `Path::of` has a special overload that does the right thing for us.
>> 
>> ----------------
>> 
>> Commits:
>>  - a58471b3:	Get path from URI to avoid getting spurious leading path separator on Windows
>> 
>> Pull request:
>> http://git.openjdk.java.net/skara/pull/36
>> 
>> Webrev:
>> https://openjdk.github.io/cr/skara/36/webrev.00
>> 
>> Patch:
>> http://git.openjdk.java.net/skara/pull/36.diff
>> 
>> Fetch command:
>> git fetch https://github.com/openjdk/skara.git pull/36/head:pull/36
> 
> Guess it doesn't recognize me as a committer yet :)
> 
> PR: http://git.openjdk.java.net/skara/pull/36

@JornVernee You will be considered committer when https://openjdk.java.net/census is updated :+1:

PR: http://git.openjdk.java.net/skara/pull/36


More information about the skara-dev mailing list