Re: RFR (XS): 8223718: Checks in check_slot_type_no_lvt() should be always executed

[gary.ad...@oracle.com]

Be sure to check copyright dates and line lengths.

On 5/29/19 6:24 PM, serguei.spit...@oracle.com wrote:

Please, review fix for:
  https://bugs.openjdk.java.net/browse/JDK-8223718

Webrev:

http://cr.openjdk.java.net/~sspitsyn/webrevs/2019/8223718-jvmti-getlocal.1/

Summary:
  The JVMTI GetLocal<TYPE>() does not return the error code
  JVMTI_ERROR_INVALID_SLOT for the T_CONFLICT if the LVT is present.
  The fix is to always execute the check_slot_type_no_lvt().
  The test nsk/jvmti/unit/GetLocalVariable/getlocal003 is updated
  to expect the error code JVMTI_ERROR_INVALID_SLOT additionally to
  the JVMTI_ERROR_TYPE_MISMATCH in a couple of cases.

Testing:
  Successful execution of the nsk.jvmti tests (release/fastdebug).
  Mach5 run is in progress.

Thanks,
Serguei