Looks good!

Thanks,
/Staffan


> On 19 nov 2014, at 15:28, Markus Grönlund <[email protected]> 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 
> <https://bugs.openjdk.java.net/browse/JDK-8065361>
> Webrev: http://cr.openjdk.java.net/~mgronlun/8065361/webrev01/ 
> <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

Reply via email to