RFR(XS) 8183935: G1GCPhaseTimes and G1RootProcessor do not include macros.hpp
mikael.gerdin at oracle.com
Mon Jul 17 14:22:47 UTC 2017
On 2017-07-17 16:21, Thomas Schatzl wrote:
> Hi Mikael,
> On Mon, 2017-07-17 at 16:12 +0200, Mikael Gerdin wrote:
>> Please review this trivial change to add includes of macros.hpp to
>> G1GCPhaseTimes and G1RootProcessor. They both the value of
>> and as such should explicitly include the proper header to ensure
>> it is set to the correct value.
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8183935
>> Webrev: http://cr.openjdk.java.net/~mgerdin/8183935/webrev.0/
>> Testing: JPRT build-only
> ship it :) Although I do not think it is necessary to include
> macros.hpp both in the hpp and cpp file, but not sure. It won't hurt.
I sort of agree but I think it's a nice convention to always #include
macros in files which look at the INCLUDE_* macros.
Thanks for the review!
More information about the hotspot-gc-dev