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

Stefan Karlsson stefan.karlsson at oracle.com
Fri Dec 18 12:02:26 UTC 2015


Looks good.

Thanks,
StefanK

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/
> Testing:
> local compilation
>
> Thanks,
>    Thomas
>
>




More information about the hotspot-gc-dev mailing list