RFR: 8072910: Move the g1EvacFailure.hpp implementation to g1EvacFailure.cpp
Thomas Schatzl
thomas.schatzl at oracle.com
Wed Feb 11 13:01:35 UTC 2015
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,
Thomas
More information about the hotspot-gc-dev
mailing list