RFR: 8355443: [java.io] Use @requires tag instead of exiting based on File.separatorChar value

[Brian Burkhalter]

On Thu, 24 Apr 2025 16:41:27 GMT, Naoto Sato <naoto at openjdk.org> wrote:

>> For tests of the `java.io` package, instead of doing this
>> 
>>     public static void main(String[] args) throws Exception {
>>         if (File.separatorChar != '\') {
>>             /* This test is only valid on win32 systems */
>>             return;
>>         }
>> 
>> do this
>> 
>>    @requires (os.family == "windows")
>
> test/jdk/java/io/pathNames/win32/SJIS.java line 51:
> 
>> 49:            that use the SJIS encoding */
>> 50:         String enc = System.getProperty("file.encoding");
>> 51:         if ((enc == null) || !enc.equals("SJIS")) return;
> 
> I just wonder this test has ever run since JDK18, as file.encoding is always UTF-8 unless COMPAT is specified (and this test case does not specify it)

So do you think that this test should be removed? If so, I think that would be a different PR. Thanks.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24838#discussion_r2058870526