RFR: 8183124: Remove OopsInHeapRegionClosure

Erik Helin erik.helin at oracle.com
Wed Jun 28 14:20:58 UTC 2017


On 06/28/2017 03:36 PM, Stefan Johansson wrote:
>
>
> On 2017-06-28 15:30, Erik Helin wrote:
>> ...and now with subject as well :)
>>
>> Erik
>>
>> On 06/28/2017 02:59 PM, Erik Helin wrote:
>>> Hi all,
>>>
>>> this small patch removes the class OopsInHeapRegionClosure.
>>> OopsInHeapRegionClosure only contains a protected _from field and the
>>> public method set_from, and there are only two other classes inheriting
>>> from OopsInHeapRegionClosure (G1ScanClosureBase and UpdareRsetDeferred).
>>>
>>> This patch gets rid of the class OopsInHeapRegionClosure and adds the
>>> corresponding field and method to the classes inheriting from
>>> OopsInHeapRegionClosure.
>>>
>>> Patch: http://cr.openjdk.java.net/~ehelin/8183124/00/
>>>
> Looks good,

Thanks Stefan, appreciate the quick review!

Erik

> StefanJ
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8183124
>>>
>>> Testing: make jprt
>>>
>>> Thanks,
>>> Erik
>



More information about the hotspot-gc-dev mailing list