RFR (S): 8232777: Rename G1Policy::_max_rs_length as it is no maximum

[Stefan Johansson]

On 2019-10-22 21:16, Kim Barrett wrote:
>> On Oct 22, 2019, at 2:02 PM, Thomas Schatzl <thomas.schatzl at oracle.com> wrote:
>>
>> Hi all,
>>
>>   can I have reviews for this small cleanup that renames G1Policy::_max_rs_length to just _rs_length because the contained value is simply no maximum. This causes some confusion down the line in its use (imo).
>>
>> CR:
>> https://bugs.openjdk.java.net/browse/JDK-8232777
>> Webrev:
>> http://cr.openjdk.java.net/~tschatzl/8232777/webrev/
>> Testing:
>> local compilation
>>
>> Thanks,
>>   Thomas
> 
> You missed one in a comment:
> src/hotspot/share/gc/g1/g1Policy.cpp
>   757     // This is defensive. For a while _max_rs_length could get
> 
> Otherwise than that, looks good, and trivial.
> 
Look good,
Stefan