On Fri, 29 Apr 2022 14:16:54 GMT, Aleksei Efimov <aefi...@openjdk.org> wrote:

>> Michael Felt has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   missing semi-colon
>
> The changes looks ok to me given that the test can be compiled and executed 
> with no failures. 
> The following question being asked before ,but is there a reason why we check 
> platform in the test code, instead of using [requires jtreg 
> tag](https://openjdk.java.net/jtreg/tag-spec.html#requires_names) ?  
> The following requires tag could be used as a replacement for the proposed 
> change:
> 
> * @requires (os.family != "windows" & os.family != "aix")

@AlekseiEfimov , @dfuch - please review again.

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

PR: https://git.openjdk.java.net/jdk/pull/7013

Reply via email to