On Wed, 14 Sep 2022 05:50:32 GMT, Serguei Spitsyn <sspit...@openjdk.org> 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

Reply via email to