Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8199561

a couple of typos in the JVMTI spec

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • P4
    • 11
    • 9, 10, 11
    • hotspot
    • None
    • b07

    Description

      Below is a email from Jc Beyler with the suggested fix:

      Hi all,

      I saw an error in the SetEventNotificationMode method where the parameter is called event_thread but the documentation was referring to it as thread. I then went and did a quick scan of the documentation and found one type of "couse" instead of "course".

      Here is the diff, not sure it was worth doing a webrev for it but let me know:
      diff -r 2d1d0c66966b src/hotspot/share/prims/jvmti.xml
      --- a/src/hotspot/share/prims/jvmti.xml Mon Mar 12 14:11:54 2018 -0700
      +++ b/src/hotspot/share/prims/jvmti.xml Tue Mar 13 10:35:03 2018 -0700
      @@ -693,7 +693,7 @@
           mechanism causes the unload (an unload mechanism is not specified in this document)
           or the library is (in effect) unloaded by the termination of the VM whether through
           normal termination or VM failure, including start-up failure.
      - Uncontrolled shutdown is, of couse, an exception to this rule.
      + Uncontrolled shutdown is, of course, an exception to this rule.
           Note the distinction between this function and the
           <eventlink id="VMDeath">VM Death event</eventlink>: for the VM Death event
           to be sent, the VM must have run at least to the point of initialization and a valid
      @@ -9405,7 +9405,7 @@
        the event <paramlink id="event_type"></paramlink> will be disabled
        </constant>
        </constants>
      - If <code>thread</code> is <code>NULL</code>,
      + If <code>event_thread</code> is <code>NULL</code>,
        the event is enabled or disabled globally; otherwise, it is
        enabled or disabled for a particular thread.
        An event is generated for

      Thanks,
      Jc

      Attachments

        Activity

          People

            jcbeyler Jean Christophe Beyler
            sspitsyn Serguei Spitsyn
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: