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

[Markus Grönlund]

Thanks Stefan and Staffan for reviewing!

 

/Markus

 

From: Staffan Larsen 
Sent: den 19 november 2014 15:56
To: Markus Grönlund
Cc: serviceability-dev at openjdk.java.net; Stefan Karlsson
Subject: Re: RFR(S): 8065361: Fixup headers and definitions for INCLUDE_TRACE

 

Looks good!

 

Thanks,

/Staffan

 

 

On 19 nov 2014, at 15:28, Markus Grönlund <HYPERLINK "mailto:markus.gronlund at oracle.com"markus.gronlund at oracle.com> 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/

 

Description:

#include "utilities/macros.hpp" for every compilation unit using INCLUDE_TRACE. Header include guards comments updated to C++ style.

 

Thanks in advance

Markus

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