On Sun, 11 Oct 2020 22:21:28 GMT, Ioi Lam <ik...@openjdk.org> wrote: > This PR changes declarations in jvmti.h like the following from > > struct _jvmtiTimerInfo; > typedef struct _jvmtiTimerInfo jvmtiTimerInfo; > > to > > struct jvmtiTimerInfo; > typedef struct jvmtiTimerInfo jvmtiTimerInfo; > > This way, it becomes possible to make forward declaration in C++ code like > this, without assuming the knowledge of the > `struct _jvmtiTimerInfo` type: > struct jvmtiTimerInfo; > > Please see bug report > [JDK-8254364](https://bugs.openjdk.java.net/browse/JDK-8254364) for the > before/after versions of > jvmti.h, which is generated from XML. > Tested with build tiers 1-5 in Mach5.
Hi Ioi, This change looks good to me. Thanks, Serguei ------------- Marked as reviewed by sspitsyn (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/596
