Hi Phil, The changes look fine. Although, in C++, it is recommended to use static_cast/dynamic_cast/reinterpret_cast rather than plain old C style cast.
Thanks, Krishna -----Original Message----- From: Phil Race Sent: Thursday, March 15, 2018 3:17 AM To: swing-dev@openjdk.java.net Subject: <Swing Dev> RFR: 8198897: Compilation errors in jdk.accessibility with VS 2017 bug:https://bugs.openjdk.java.net/browse/JDK-8198897 webrev: http://cr.openjdk.java.net/~prr/8198897/ Small fixes in the accessibility native code. See the bug report for the warnings. -phil.