RFR (XXXS): 8145752: Fix include guards in GC code

Thomas Schatzl thomas.schatzl at oracle.com
Fri Dec 18 13:31:30 UTC 2015


Mikael, Stefan,

  thanks for the review. I am treating this change as a trivial change
and push it right away without waiting further reviews.

Thanks,
  Thomas

On Fri, 2015-12-18 at 13:03 +0100, Mikael Gerdin wrote:
> Thomas,
> 
> On 2015-12-18 13:00, Thomas Schatzl wrote:
> > Hi all,
> >
> >    can I have reviews for this change that fixes include guards in GC
> > code?
> >
> > In the review for 8145667 StefanK noticed that some include guards are
> >
> > #endif  /* SHARE_VM_GC_G1_G1FROMCARDCACHE_HPP */
> >
> > while they should be
> >
> > #endif // SHARE_VM_GC_G1_HEAPREGIONREMSET_HPP
> >
> > CR:
> > https://bugs.openjdk.java.net/browse/JDK-8145752
> > Webrev:
> > http://cr.openjdk.java.net/~tschatzl/8145752/webrev/
> 
> Looks good :)
> /Mikael
> 
> > Testing:
> > local compilation
> >
> > Thanks,
> >    Thomas
> >
> >
> 





More information about the hotspot-gc-dev mailing list