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

[Thomas Schatzl]

Hi Kim, Stefan,

   thanks for your reviews.

For reference, I updated the webrev in place.

Thanks,
   Thomas

On 23.10.19 08:16, Stefan Johansson wrote:
> 
> 
> 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