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

David Holmes david.holmes at oracle.com
Thu Dec 8 08:23:10 UTC 2016


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