On Wed, 14 Sep 2022 05:50:32 GMT, Serguei Spitsyn <[email protected]> wrote:
>> Kevin Walls has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Clarify that loop is for checking heap not changing. Exception if
>> continually changing.
>
> test/hotspot/jtreg/serviceability/tmtools/jstat/utils/GcProvoker.java line 76:
>
>> 74: System.gc();
>> 75: continue;
>> 76: } else {
>
> In fact, the `else-statement` is not needed here because the `if-statement`
> is ended with `continue`.
> At least, this looks confusing to me.
OK sure, just updated to remove else, and add a comment which may help.
-------------
PR: https://git.openjdk.org/jdk/pull/10218
