RFR: 8306440: Rename PSS:_num_optional_regions to _num_initial_optional_regions

Albert Mingkun Yang ayang at openjdk.org
Fri Apr 21 09:26:43 UTC 2023


On Wed, 19 Apr 2023 11:13:36 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

> Hi all,
> 
>    please review this small change that clarifies the use of `PSS::_num_optional_regions` to be the initial amount of optional regions, not the current amount of optional regions.
> 
> Testing: gha
> 
> Thanks,
>   Thomas

I feel `_max_num_optional_regions` better reflects its intrinsic semantic.

Inside young-gc pause, `initial` and `optional` most often refer to the dual initial-evac vs optional-evac and everything associated with them. Therefore, it can be confusing to see both words in one name.

(Ofc, this is quite subjective.)

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

Marked as reviewed by ayang (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/13532#pullrequestreview-1395426411


More information about the hotspot-gc-dev mailing list