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