Looks good

Erik

Markus Grönlund skrev 2014-12-17 15:45:
Greetings,

Kindly asking for reviews for the following changeset:

Bug: https://bugs.openjdk.java.net/browse/JDK-8066814

Webrev: http://cr.openjdk.java.net/~mgronlun/8066814/webrev01/

Description:

TraceEvent currently exposes internals unnecessarily.

Therefore:

Remove unnecessarily exposed methods.
Add assert for not committing a cancelled event.
Add method stubs for !INCLUDE_TRACE

Thanks in advance

Markus

Reply via email to