Integrated: 8336675: Remove UnixFileSystemProvider.checkPath in favor of UnixPath.toUnixPath

Andrey Turbanov aturbanov at openjdk.org
Fri Jul 19 05:57:44 UTC 2024


On Fri, 12 Jul 2024 11:54:15 GMT, Andrey Turbanov <aturbanov at openjdk.org> wrote:

> Implementation of both methods is the same.
> https://github.com/openjdk/jdk/blob/9b6f6c5c9dd6d0fbb056e8d84c3a0888a3320edf/src/java.base/unix/classes/sun/nio/fs/UnixFileSystemProvider.java#L129-L135
> 
> https://github.com/openjdk/jdk/blob/9b6f6c5c9dd6d0fbb056e8d84c3a0888a3320edf/src/java.base/unix/classes/sun/nio/fs/UnixPath.java#L172-L179
> 
> `UnixPath.toUnixPath` is already used by UnixFileSystemProvider in multiple places. Two more usages shouldn't cause problems.
> 
> Tested on linux x64 `test/jdk/java/nio` + `test/jdk/java/io`

This pull request has now been integrated.

Changeset: f5871df2
Author:    Andrey Turbanov <aturbanov at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/f5871df25c60990825babb3bbae38c5ade93f097
Stats:     21 lines in 1 file changed: 0 ins; 17 del; 4 mod

8336675: Remove UnixFileSystemProvider.checkPath in favor of UnixPath.toUnixPath

Reviewed-by: jpai, alanb, bpb

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

PR: https://git.openjdk.org/jdk/pull/20155


More information about the nio-dev mailing list