Re: 8164523: Clean up metadata for event based tracing

2016-08-20 Thread Claes Redestad
+1 /Claes On 2016-08-20 20:49, Erik Gahlin wrote: Hi, Could I have review of this fix to the event tracing metadata. The patch touches many files, but it doesn't change the control flow, only class and field names. Webrev: http://cr.openjdk.java.net/~egahlin/8164523/ Bug: https://bugs.openjd

RE: 8164523: Clean up metadata for event based tracing

2016-08-22 Thread Markus Gronlund
Hi Erik, This looks good, thanks for cleaning this up. Thanks Markus -Original Message- From: Erik Gahlin Sent: den 20 augusti 2016 20:49 To: serviceability-dev@openjdk.java.net Subject: 8164523: Clean up metadata for event based tracing Hi, Could I have review of this fix to the even

Re: 8164523: Clean up metadata for event based tracing

2016-08-22 Thread Marcus Larsson
Hi, On 08/20/2016 08:49 PM, Erik Gahlin wrote: Hi, Could I have review of this fix to the event tracing metadata. The patch touches many files, but it doesn't change the control flow, only class and field names. Webrev: http://cr.openjdk.java.net/~egahlin/8164523/ Looks good to me! Than