On Wed, 7 Sep 2022 05:51:53 GMT, Serguei Spitsyn <sspit...@openjdk.org> wrote:

> The JDI files `com/sun/jdi/ClassLoaderReference.java` and 
> `com/sun/jdi/VirtualMachine.java` have incorrect links to the JVM TI spec 
> with the extra folder name `jvmti`. 
> 
> The fix is to get rid unneeded part from the links:
> 
> --- a/src/jdk.jdi/share/classes/com/sun/jdi/ClassLoaderReference.java
> +++ b/src/jdk.jdi/share/classes/com/sun/jdi/ClassLoaderReference.java
> @@ -93,7 +93,7 @@ public interface ClassLoaderReference extends 
> ObjectReference {
>       * classes which this class loader can find by name. The list
>       * has length 0 if no classes are visible to this classloader.
>       *
> -     * @see <a 
> href="{@docRoot}/../specs/jvmti/jvmti.html#GetClassLoaderClasses">
> +     * @see <a href="{@docRoot}/../specs/jvmti.html#GetClassLoaderClasses">
>       *     JVM TI GetClassLoaderClasses</a>
>       */
>      List<ReferenceType> visibleClasses();
> diff --git a/src/jdk.jdi/share/classes/com/sun/jdi/VirtualMachine.java 
> b/src/jdk.jdi/share/classes/com/sun/jdi/VirtualMachine.java
> index 12cdd566989..64c91c8d1bb 100644
> --- a/src/jdk.jdi/share/classes/com/sun/jdi/VirtualMachine.java
> +++ b/src/jdk.jdi/share/classes/com/sun/jdi/VirtualMachine.java
> @@ -144,7 +144,7 @@ public interface VirtualMachine extends Mirror {
>       *
>       * @return a list of {@link ReferenceType} objects, each mirroring
>       * a loaded type in the target VM.
> -     * @see <a href="{@docRoot}/../specs/jvmti/jvmti.html#GetLoadedClasses">
> +     * @see <a href="{@docRoot}/../specs/jvmti.html#GetLoadedClasses">
>       * JVM TI GetLoadedClasses</a> regarding how class and interface 
> creation can be triggered
>       */
>      List<ReferenceType> allClasses();

Marked as reviewed by alanb (Reviewer).

-------------

PR: https://git.openjdk.org/jdk/pull/10191

Reply via email to