RFR: 8262742: (fs) Add Path::resolve with varargs string [v5]
Brian Burkhalter
bpb at openjdk.org
Mon Jul 17 19:10:46 UTC 2023
On Sat, 15 Jul 2023 07:11:03 GMT, Alan Bateman <alanb at openjdk.org> wrote:
>> Brian Burkhalter has updated the pull request incrementally with one additional commit since the last revision:
>>
>> 8262742: Improve verbiage per reviewer PR comments
>
> src/java.base/share/classes/java/nio/file/Path.java line 527:
>
>> 525: * {@code more} specifies one or more {@code Path}s, then each non-empty
>> 526: * {@code Path}, including {@code first}, is resolved iteratively against
>> 527: * this {@code Path}.
>
> The second paragraph has the same issue as before, it says "resolved iteratively against this Path" but that isn't correct. Same issue in the description of the "more" method. The first paragraph is correct, and I think we'll need to re-phrase the second paragraph to accurate specify how the iterative resolution works.
0fa126fd2d3861c56b85093e8a2483afa1ad9ef6 attempts to improve the aforementioned verbiage.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14805#discussion_r1265788140
More information about the nio-dev
mailing list