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