Thank you, Chris!
Serguei
On 11/20/19 18:22, Chris Plummer wrote:
+1
On 11/20/19 3:32 PM, Alex Menkov wrote:
Looks good.
--alex
On 11/19/2019 15:36, [email protected] wrote:
Please, review a fix for:
https://bugs.openjdk.java.net/browse/JDK-8169467
Webrev:
http://cr.openjdk.java.net/~sspitsyn/webrevs/2019/8169467-jvmti-local-instance.1/
Summary:
The JVMTI GetLocalInstance function should return
JVMTI_ERROR_INVALID_SLOT for static method frames.
Instead, it returns the JVMTI_ERROR_TYPE_MISMATCH error.
The fix adds necessary checks into the implementation.
Testing in progress:
- Locally on Linux-x64 with: vmTestbase_nsk_jvmti,
vmTestbase_nsk_jdi, jdk_jdi
- All mach5 hs-tier5
Thanks,
Serguei
