RFR: 8363720: Follow up to JDK-8360411 with post review comments [v3]

Brian Burkhalter bpb at openjdk.org
Tue Jul 29 18:50:56 UTC 2025


On Thu, 24 Jul 2025 10:06:39 GMT, Darragh Conway <duke at openjdk.org> wrote:

>> There were a couple of post review comments after the /integrate command was submitted. This issue will address those comments. Also created constants for hardcoded values and tests. Fixed some typos.
>
> Darragh Conway has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Fix remaining indentation errors (From my first commit on this branch)

test/jdk/java/io/File/MaxPathLength.java line 69:

> 67:     }
> 68: 
> 69:     private static void setIsWindows() {

Instead of `isWindows` and `setIsWindows`, perhaps you should consider using `jdk.test.lib.Platform.isWindows()`? See, for example, [GetXSpace.java](https://github.com/openjdk/jdk/blob/master/test/jdk/java/io/File/GetXSpace.java), lines 30-31, 46, and 109.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26445#discussion_r2240679861


More information about the core-libs-dev mailing list