Hi Mandy,
I'm fine with this.
Roger
On 6/23/20 5:42 PM, Mandy Chung wrote:
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
