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