RFR: 8306440: Rename PSS:_num_optional_regions to _max_num_optional_regions [v3]

Fri Apr 21 14:21:01 UTC 2023

On Fri, 21 Apr 2023 14:17:01 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
>
> Thomas Schatzl has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains two commits:
> 
>  - ayang review
>  - Rename PSS:_num_optional_regions to _num_initial_optional_regions to emphasize that it's the initial amount of optional regions
>    
>    Rename the stated member to better indicate that this is the initial amount of optional
>    region at the start of gc vs. the one for the current iteration of the evacuation.
>    Optional.

Marked as reviewed by iwalulya (Reviewer).

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

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