On 9/10/2018 4:08 PM, Chris Plummer wrote:
Looks good. Any idea what caused it, other than it was introduced by
JDK-8209611?
It wasn't in the original webrev for 8209611 (no cast at all) so I'm
guessing it was a last minute change that somehow went awry. I wonder if
compilers have a -Wunnecessary_cast warning that may have caught it ;-)
David
Chris
On 10/8/18 8:54 PM, JC Beyler wrote:
Hi all,
Seems like there is an over-zealous casting in the file
nsk/jvmti/scenarios/events/EM06/em06t001/em06t001.cpp:
jclassName = (jstring) (jstring) (jstring) (jstring) (jstring)
(jstring) (jstring) (jstring) (jstring)
NSK_CPP_STUB3(CallObjectMethod, jni_env, klass,
This webrev fixes that:
Webrev: http://cr.openjdk.java.net/~jcbeyler/8211905/webrev.00/
<http://cr.openjdk.java.net/%7Ejcbeyler/8211905/webrev.00/>
Bug: https://bugs.openjdk.java.net/browse/JDK-8211905
Thanks,
Jc
