Integrated: 8254364: Remove leading _ from struct/union declarations in jvmti.h
Ioi Lam
iklam at openjdk.java.net
Tue Oct 13 05:26:12 UTC 2020
On Sun, 11 Oct 2020 22:21:28 GMT, Ioi Lam <iklam at 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.
This pull request has now been integrated.
Changeset: c9ca1bb0
Author: Ioi Lam <iklam at openjdk.org>
URL: https://git.openjdk.java.net/jdk/commit/c9ca1bb0
Stats: 7 lines in 1 file changed: 0 ins; 0 del; 7 mod
8254364: Remove leading _ from struct/union declarations in jvmti.h
Reviewed-by: dholmes, sspitsyn
-------------
PR: https://git.openjdk.java.net/jdk/pull/596
More information about the serviceability-dev
mailing list