RFR: 8357193: [VS 2022 17.14] Warning C5287 in debugInit.c: enum type mismatch during build
Serguei Spitsyn
sspitsyn at openjdk.org
Mon May 19 17:56:52 UTC 2025
On Mon, 19 May 2025 05:26:36 GMT, Sergey Bylokhov <serb at openjdk.org> wrote:
> I can suppress it by extracting JVMTI_VERSION into local variable similar to how other code looks like:
>
> jint version = JVMTI_VERSION;
> jvmtiCompileTimeMajorVersion = ( version & JVMTI_VERSION_MASK_MAJOR )
> >> JVMTI_VERSION_SHIFT_MAJOR;
> jvmtiCompileTimeMinorVersion = ( version & JVMTI_VERSION_MASK_MINOR )
> >> JVMTI_VERSION_SHIFT_MINOR;
> jvmtiCompileTimeMicroVersion = ( version & JVMTI_VERSION_MASK_MICRO )
> >> JVMTI_VERSION_SHIFT_MICRO;
>
> but it might be better just suppress it in the make file?
I like the suggestion to extract JVMTI_VERSION into a local variable. Fixing it in the make file is fine too.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/25293#issuecomment-2891829112
More information about the serviceability-dev
mailing list