RFR: 8072910: Move the g1EvacFailure.hpp implementation to g1EvacFailure.cpp
Stefan Karlsson
stefan.karlsson at oracle.com
Wed Feb 11 12:53:21 UTC 2015
Hi Thomas,
On 2015-02-11 11:49, Thomas Schatzl wrote:
> Hi Stefan,
>
> On Wed, 2015-02-11 at 10:40 +0100, Stefan Karlsson wrote:
>> Hi all,
>>
>> Please, review this patch to move the implementation of
>> g1EvacFailure.hpp into a new cpp file. This helps reducing our include
>> dependencies in the header files.
>>
>> 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
Thanks,
StefanK
>
> Thanks,
> Thomas
>
>
More information about the hotspot-gc-dev
mailing list