RFR: 8262742: (fs) Add Path::resolve with varargs string [v2]

Brian Burkhalter bpb at openjdk.org
Thu Jul 20 03:55:08 UTC 2023


On Mon, 17 Jul 2023 21:16:37 GMT, Brian Burkhalter <bpb at openjdk.org> wrote:

>> Multiple directories would be good but I don't know about `..`.
>
>> Should there be test cases that include multiple directories in a path: "abc/def/xyz"?
> 
> Added some multiple directory cases in 505d72b48c5d34bbf4474c6fed43e2f269e8e135.
> 
>> Also cases with ".."
> 
> This is not relevant. For example, using the existing `resolve(Path)`:
> 
> 
> jshell> java.nio.file.Path.of("/tmp").resolve("..").resolve("..")
> $1 ==> /tmp/../..

Addressed in c2a8faaa1b81dbb40e1a5989a02c9e5935417358.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14805#discussion_r1268901442


More information about the nio-dev mailing list