RFR (XXS): 8153193: Missing includes in gc/g1/heapRegionBounds.hpp (original) (raw)
Thomas Schatzl thomas.schatzl at oracle.com
Thu Mar 31 13:10:41 UTC 2016
- Previous message (by thread): RFR (S): 8135200: Add White Box method that enumerates G1 old regions with less than specified liveness and collects statistics.
- Next message (by thread): RFR (XXS): 8153193: Missing includes in gc/g1/heapRegionBounds.hpp
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Hi all,
can I have quick reviews for this change that adds a missing include to the file in question?
The HeapRegionBounds class inherits from AllStatic, which is in memory/allocation.hpp that is not included. Some future changes break because of this oversight.
For current sources there is no difference.
Note that strictly we should probably also include stddef.h for size_t, but looking through other sources, this is (almost) never done. So I refrained from doing so here too.
CR: https://bugs.openjdk.java.net/browse/JDK-8153193 Webrev: http://cr.openjdk.java.net/~tschatzl/8153193/webrev/ Testing: local testing
Thanks, Thomas
- Previous message (by thread): RFR (S): 8135200: Add White Box method that enumerates G1 old regions with less than specified liveness and collects statistics.
- Next message (by thread): RFR (XXS): 8153193: Missing includes in gc/g1/heapRegionBounds.hpp
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]