RFR(S): 8065361: Fixup headers and definitions for INCLUDE_TRACE

Stefan Karlsson stefan.karlsson at oracle.com
Wed Nov 19 14:37:19 UTC 2014


On 2014-11-19 15:28, Markus Grönlund wrote:
>
> Greetings,
>
> Kindly asking for reviews for the following changeset to clean up the 
> usage of the INCLUDE_TRACE define:
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8065361
>
> Webrev: http://cr.openjdk.java.net/~mgronlun/8065361/webrev01/ 
> <http://cr.openjdk.java.net/%7Emgronlun/8065361/webrev01/>
>
> Description:
>
> #include ”utilities/macros.hpp” for every compilation unit using 
> INCLUDE_TRACE. Header include guards comments updated to C++ style.
>

Looks good.

Thanks,
StefanK

> Thanks in advance
>
> Markus
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20141119/1b9fa89d/attachment-0001.html>


More information about the serviceability-dev mailing list