RFR: 8262742: (fs) Add Path::resolve with varargs string [v5]
Alan Bateman
alanb at openjdk.org
Tue Jul 18 17:18:14 UTC 2023
On Mon, 17 Jul 2023 19:05:14 GMT, Brian Burkhalter <bpb at openjdk.org> wrote:
>> 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.
> 0fa126f attempts to improve the aforementioned verbiage.
The wording in the latest (db4d9bd6) looks good.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14805#discussion_r1267081744
More information about the nio-dev
mailing list