On Fri, 26 Jun 2026 11:30:54 GMT, Ivan Walulya <[email protected]> wrote:

>> src/hotspot/share/gc/g1/g1HeapRegionSet.cpp line 300:
>> 
>>> 298:   assert(num_regions() + num_regions_to_remove == old_length,
>>> 299:          "[%s] new length should be consistent "
>>> 300:          "new length: %u old length: %u num_regions: %u",
>> 
>> length -> old_num_regions etc.?
>
> left length naming for the explicit list `G1FreeRegionList`.

The num_regions: should probably be changed now that things have been renamed.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/31688#discussion_r3721468808

Reply via email to