On Fri, 25 Apr 2025 16:50:42 GMT, Brian Burkhalter <b...@openjdk.org> wrote:
>> Use the `@requires` tag instead of obtaining the operating system name from
>> the `os.name` property and then exiting if the test is not run on that
>> operating system.
>
> Brian Burkhalter has updated the pull request incrementally with one
> additional commit since the last revision:
>
> 8355444: Revert addition of @requires to MacPathTest and add a comment
test/jdk/java/io/FileOutputStream/ManyFiles.java line 42:
> 40: public static void main(String args[]) throws Exception {
> 41: // Vestigial comment from before @requires tag was added:
> 42: // Linux does not yet allow opening this many files; Solaris
Nit - If you do happen to update this PR for any other reason, then maybe
consider adding jtreg's `@comment` in the test definition
(https://openjdk.org/jtreg/tag-spec.html#INFORMATIONAL_TAGS) just before the
`@requires` line and move this existing comment there.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24860#discussion_r2064043079
