RFR(S): 7034834 JVMTI xml file referring to old JDK version

[serguei.spitsyn at oracle.com]

Dmitry,

Thank you a lot for review!
I will fix issues pointed below.
Nice catch.

Thanks,
Serguei


On 12/7/16 06:41, Dmitry Samersoff wrote:
> Serguei,
>
> Looks good to me.
>
> PS:
>   Could you add spaces around -- in VM--see at l.2112
>   Also, Only one mention of JNI Specification of 3 is clickable.
>
> -Dmitry
>
>
> On 2016-12-06 21:37, serguei.spitsyn at oracle.com wrote:
>> Dmitry, David and Coleen,
>>
>>
>> The updated webrev is:
>> http://cr.openjdk.java.net/~sspitsyn/webrevs/2016/hotspot/7034834-Doc.hs2/
>>
>> The jvmti_html.zip is in the attachement.
>>
>> The change is that the prefix
>> http://docs.oracle.com/javase/<majorversion> is added
>> in the <xsl:template match="externallink">.
>> This prefix is tripped in the jvmti.xml from all the externallink id's.
>>
>> Now, the JDK version in the link will be automatically updated with the
>> majorversion change.
>>
>>
>> Thanks,
>> Serguei
>>
>>
>>
>> On 12/5/16 21:30, serguei.spitsyn at oracle.com wrote:
>>> Hi Coleen,
>>>
>>> Thank you for looking at this fix!
>>>
>>>
>>> On 12/5/16 07:41, Coleen Phillimore wrote:
>>>> I checked your jdk9 links to see if they correspond to jdk8 links and
>>>> they all exist for jdk8.
>>> Right. I've verified that too.
>>>
>>>> Is there any way to parameterize this in jvmti.xml?
>>> I was already thinking about it and only after your question got an
>>> idea. :)
>>> Let me check if it can be simple enough in the .xsl implementation.
>>>
>>>
>>>> http://docs.oracle.com/javase/9/docs/technotes/guides
>>>>
>>>> Otherwise, this looks good.
>>> Thanks!
>>> Serguei
>>>
>>>
>>>> thanks,
>>>> Coleen
>>>>
>>>>
>>>> On 12/3/16 2:01 AM, serguei.spitsyn at oracle.com wrote:
>>>>> One more attempt to send this e-mail correctly. :)
>>>>>
>>>>> Please, review a fix for the bug:
>>>>>    https://bugs.openjdk.java.net/browse/JDK-7034834
>>>>>
>>>>> Webrev:
>>>>> http://cr.openjdk.java.net/~sspitsyn/webrevs/2016/hotspot/7034834-Doc.hs1/
>>>>>
>>>>>
>>>>>
>>>>> Summary:
>>>>>
>>>>>    The JVM TI spec has links to the old JDK 7 release documents.
>>>>>    The fix is to make it up-to-date and link to the JDK 9 docs.
>>>>>
>>>>>    One issue with the fix that the JDK 9 doc pages do not exist yet.
>>>>>    I hope that the JDK 9 release doc pages will be created the same way
>>>>>    as they were for JDK 8. The JVM TI spec for JDK 8 has the same
>>>>>    problem as it also has links to the JDK 7 release documents.
>>>>>    I'd like to fix this issue in the JDK 9 ASAP in order to be able
>>>>>    to backport the fix to JDK 8.
>>>>>
>>>>>
>>>>> Thanks,
>>>>> Serguei
>>>>>
>