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