[15] RFR JDK-8247785: Small clarification to the javadoc about builtin class loaders

[Mandy Chung]

On 6/23/20 12:01 PM, Roger Riggs wrote:
> Hi Mandy,
>
> There may be a missing "to" in:
>
> + *     <em>Platform classes</em> are visible the platform class loader
> ++ *     <em>Platform classes</em> are visible *via* the platform 
> class loader
>

I caught this accidental change too.

> The second change seems to be self referential using "parent" to 
> define itself.
>
> And pre-existing in the description of getSystemClassLoader:
>
> * The platform class loader is a parent or an ancestor of the system 
> class * loader that all platform classes are visible to it.
>
> Is missing "so" in :
>
> * loader so that all platform classes are visible to it.
>
> Both paragraphs are difficult to read and understand. (  I think the 
> originals are more readable).

I made a minor adjustment to break the sentence into two.  That should help.

diff --git a/src/java.base/share/classes/java/lang/ClassLoader.java 
b/src/java.base/share/classes/java/lang/ClassLoader.java
--- a/src/java.base/share/classes/java/lang/ClassLoader.java
+++ b/src/java.base/share/classes/java/lang/ClassLoader.java
@@ -133,8 +133,9 @@
   *     It is the virtual machine's built-in class loader, typically 
represented
   *     as {@code null}, and does not have a parent.</li>
   * <li><p>{@linkplain #getPlatformClassLoader() Platform class loader}.
- *     All <em>platform classes</em> are visible to the platform class 
loader
- *     that can be used as the parent of a {@code ClassLoader} instance.
+ *     <em>Platform classes</em> are visible to the platform class loader.
+ *     The platform class loader can be used as the parent of a {@code 
ClassLoader}
+ *     instance.
   *     Platform classes include Java SE platform APIs, their 
implementation
   *     classes and JDK-specific run-time classes that are defined by the
   *     platform class loader or its ancestors.
@@ -152,7 +153,7 @@
   *     The system class loader is typically used to define classes on the
   *     application class path, module path, and JDK-specific tools.
   *     The platform class loader is a parent or an ancestor of the 
system class
- *     loader that all platform classes are visible to it.</li>
+ *     loader.  It searches and loads the platform classes through its 
parent.</li>
   * </ul>
   *
   * <p> Normally, the Java virtual machine loads classes from the local 
file

Thanks
Mandy