On Tue, 29 Apr 2025 05:39:14 GMT, Alan Bateman <al...@openjdk.org> wrote:
>> Brian Burkhalter has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8355444: Add comment tag tp ManyFiles > > test/jdk/java/io/FileOutputStream/ManyFiles.java line 33: > >> 31: * Windows capability it is much simpler to only run it >> 32: * on that platform. >> 33: * @requires (os.family != "linux") > > I think this comment should be replaced with a clear summary as to why three > is a requires. No need to mention Solaris. Perhaps the `@requires` tag is no longer needed. Without it, this test passed 30 repeats in the CI on both Intel and ARM Linux nodes. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/24860#discussion_r2066952759
