RFR: 8072910: Move the g1EvacFailure.hpp implementation to g1EvacFailure.cpp

Stefan Karlsson stefan.karlsson at oracle.com
Wed Feb 11 13:51:55 UTC 2015


On 2015-02-11 14:01, Thomas Schatzl wrote:
> Hi Stefan,
>
> On Wed, 2015-02-11 at 13:53 +0100, Stefan Karlsson wrote:
>> Hi Thomas,
>>
>> [...]
>>>> http://cr.openjdk.java.net/~stefank/8072910/webrev.00/
>>>> https://bugs.openjdk.java.net/browse/JDK-8072910
>>>     is it possible to fix the comment about "Closures and tasks associated
>>> with any self-forwarding pointers..." at the top of the hpp file?
>>>
>>> There is only one task and no closures now.
>>>
>>> Not sure if that same comment is required in the cpp file. Maybe remove
>>> it.
>>>
>>> Otherwise looks good.
>> I've update the patch:
>>    http://cr.openjdk.java.net/~stefank/8072910/webrev.01
>>    http://cr.openjdk.java.net/~stefank/8072910/webrev.01.delta
> Looks good. Thanks for the change.

Thanks,
StefanK

>
> Thanks,
>    Thomas
>
>



More information about the hotspot-gc-dev mailing list