The fix looks good to me.

--alex

On 06/04/2019 16:30, serguei.spit...@oracle.com wrote:
Please, review a fix for the JVMTI spec bug:
https://bugs.openjdk.java.net/browse/JDK-8205126

Webrev:
http://cr.openjdk.java.net/~sspitsyn/webrevs/2019/8205126-jvmti-spec-popframe.1/


Related CSR:
https://bugs.openjdk.java.net/browse/JDK-8225142

Specdiff:
http://cr.openjdk.java.net/~sspitsyn/webrevs/2019/8205126-jvmti-spec-popframe.1/jvmti-specdiff/


Summary:
   The JVMTI PopFrame() spec does not match the implementation.
   It says the  specified thread can not be the current thread.
   The fix aligns:
    - spec with implementaion
    - PopFrame spec with ForceEarlyReturn spec

Thanks,
Serguei

Reply via email to