Request for review (JVMTI spec, small)

Keith McGuigan keith.mcguigan at oracle.com
Thu May 5 07:53:07 PDT 2011


On May 5, 2011, at 9:37 AM, Daniel D. Daugherty wrote:

> On 5/4/2011 7:10 PM, Keith McGuigan wrote:
>> Hello,
>>
>> There has been a request for us to not link directly to the JVMS  
>> from the JVMTI spec (especially using the old location in the  
>> java.sun.com domain, which will likely go away someday).  This  
>> change removes the hyperlinks to the JVMS from the JVMTI  
>> Specification and replaces it with the proper legal name and (when  
>> applicable) the chapter number referred to.  I would appreciate any  
>> reviews.
>>
>> Webrev: http://cr.openjdk.java.net/~kamg/7033669/webrev.00/
>
> src/share/vm/prims/jvmti.xml
>
>   Line 3575: "See description in" would read better as
>       "See the description in"
>
>   Line 6437: indent is off/missing.
>
>   Line 8128: indent is off/missing.
>
> src/share/vm/prims/jvmti.xsl
>   No comments.
>

Fixed, thanks!

--
- Keith


More information about the serviceability-dev mailing list