RFR: 8202863: Rename OopStorage inner collection classes

Kim Barrett kim.barrett at oracle.com
Tue May 22 05:41:47 UTC 2018


> On May 19, 2018, at 1:14 AM, coleen.phillimore at oracle.com wrote:
> 
> 
> Also, this looks trivial enough to push with one reviewer.

Thanks, I was thinking that too, since it’s a purely mechanical renaming of non-public classes,
but forgot to mention that possibility in the RFR email.  Might push today.

> Coleen
> 
> On 5/17/18 5:34 PM, Kim Barrett wrote:
>>> On May 17, 2018, at 4:44 PM, coleen.phillimore at oracle.com wrote:
>>> 
>>> Looks good!
>>> Coleen
>> Thanks.
>> 
>>> On 5/17/18 4:33 PM, Kim Barrett wrote:
>>>> [Resending hotspot-dev too, so Coleen will see it, since she requested this change.]
>>>> 
>>>> Please review this simple renaming of some private inner classes of
>>>> OopStorage.  The renamings were suggested during the review of
>>>> JDK-8200557, but were deferred to a separate change to avoid adding
>>>> complexity to that review.  The following classes are being renamed:
>>>> 
>>>> BlockArray => ActiveArray
>>>> BlockList => AllocateList
>>>> BlockEntry => AllocateEntry
>>>> 
>>>> CR:
>>>> https://bugs.openjdk.java.net/browse/JDK-8202863
>>>> 
>>>> Webrev:
>>>> http://cr.openjdk.java.net/~kbarrett/8202863/open.00/
>>>> 
>>>> Testing:
>>>> Mach5 CI equivalent.




More information about the hotspot-dev mailing list