'RFR(XS): 8221306: JVMTI spec for FramePop(), MethodExit(), and MethodEnter() could use some cleanup'

[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-serviceability-dev
Subject:    RFR(XS): 8221306: JVMTI spec for FramePop(), MethodExit(), and MethodEnter() could use some cleanup
From:       "serguei.spitsyn () oracle ! com" <serguei ! spitsyn () oracle ! com>
Date:       2020-05-31 8:11:31
Message-ID: f8731700-fc07-7df4-b405-ec3d65bbc850 () oracle ! com
[Download RAW message or body]

<html>
  <head>

    <meta http-equiv="content-type" content="text/html; charset=UTF-8">
  </head>
  <body>
    <font face="Monaco">Please, review a fix for small spec bug:<br>
        <a class="moz-txt-link-freetext" \
href="https://bugs.openjdk.java.net/browse/JDK-8221306">https://bugs.openjdk.java.net/browse/JDK-8221306</a><br>
  <br>
      Webrev:<br>
       
<a class="moz-txt-link-freetext" \
href="http://cr.openjdk.java.net/~sspitsyn/webrevs/2020/jvmt-funcs-cleanup.1/src/">http://cr.openjdk.java.net/~sspitsyn/webrevs/2020/jvmt-funcs-cleanup.1/src/</a><br>
  <br>
      Updated JVM TI spec for </font><font face="Monaco"><font
        face="Monaco">the FramePop, MethodEntry and MethodExit events</font>:<br>
       
<a class="moz-txt-link-freetext" \
href="http://cr.openjdk.java.net/~sspitsyn/webrevs/2020/jvmt-funcs-cleanup.1/docs/spec \
s/jvmti.html#FramePop">http://cr.openjdk.java.net/~sspitsyn/webrevs/2020/jvmt-funcs-cleanup.1/docs/specs/jvmti.html#FramePop</a><br>
  <br>
      Summary:<br>
        It is a minor spec cleanup for JVM TI events
      FramePop/MethodEntry/MethodExit:<br>
         - added small clarification that GetFrameLocation needs to be
      asked for frame at depth 0<br>
         - removed partly unneeded and partly incorrect statements about
      MethodExit event argument<br>
      <br>
      Testing:<br>
        Manually verified the generated jvmti.html.<br>
      <br>
      I think, there is no need to file a CSR for this spec update as it
      is just minor cleanup.<br>
      <br>
      Thanks,<br>
      Serguei<br>
    </font>
  </body>
</html>


[prev in list] [next in list] [prev in thread] [next in thread]
Configure | About | News | Add a list | Sponsored by KoreLogic