RFR: JDK-8280498: jdk/java/net/Inet4Address/PingThis.java fails on AIX [v5]

Michael Felt duke at openjdk.java.net
Sat Apr 30 10:02:51 UTC 2022


On Fri, 29 Apr 2022 14:16:54 GMT, Aleksei Efimov <aefimov at 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


More information about the net-dev mailing list