RFR(S): 8170847: Refactor trace/traceStream.hpp

Markus Gronlund markus.gronlund at oracle.com
Thu Dec 8 15:03:47 UTC 2016


Thanks David!

Markus

-----Original Message-----
From: David Holmes 
Sent: den 8 december 2016 09:23
To: Markus Gronlund; hotspot-runtime-dev at openjdk.java.net
Subject: Re: RFR(S): 8170847: Refactor trace/traceStream.hpp

Thanks for doing that Markus!

26 #include "trace/traceStream.hpp"
27 #ifdef INCLUDE_TRACE

Took me a while to realize why the include for traceStream.hpp had to be before the ifdef :)

Cheers,
David

On 8/12/2016 12:06 AM, Markus Gronlund wrote:
> Greetings,
>
>
>
> Please review refactoring of the traceStream.hpp header into an associated traceStream.cpp file:
>
>
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8170847
>
> Webrev: http://cr.openjdk.java.net/~mgronlun/8170847/webrev01/
>
>
>
> Thanks
>
> Markus
>


More information about the hotspot-runtime-dev mailing list