This is an automated email from the ASF dual-hosted git repository.
markt pushed a commit to branch 10.0.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/10.0.x by this push:
new 38aec51 Optimize the comment about class loader
38aec51 is described below
commit 38aec51e92d2077ba0483f972b77936634236558
Author: Poison <tianshuang...@gmail.com>
AuthorDate: Tue Oct 19 15:51:33 2021 +0800
Optimize the comment about class loader
In HotSpot JVM, javaseClassLoader is actually Extension ClassLoader, The
comment of javaseClassLoader mentioned:
```java
/**
* The bootstrap class loader used to load the JavaSE classes. In some
* implementations this class loader is always <code>null</code> and in
* those cases {@link ClassLoader#getParent()} will be called
recursively on
* the system class loader and the last non-null result used.
*/
private ClassLoader javaseClassLoader;
```
So I think it is more appropriate to use bootstrap classLoader in the
comments.
---
java/org/apache/catalina/loader/WebappClassLoaderBase.java | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/java/org/apache/catalina/loader/WebappClassLoaderBase.java
b/java/org/apache/catalina/loader/WebappClassLoaderBase.java
index 49e0025..5122d4e 100644
--- a/java/org/apache/catalina/loader/WebappClassLoaderBase.java
+++ b/java/org/apache/catalina/loader/WebappClassLoaderBase.java
@@ -89,7 +89,7 @@ import org.apache.tomcat.util.security.PermissionCheck;
* behavior may be completely different.
* <p>
* <strong>IMPLEMENTATION NOTE</strong> - By default, this class loader follows
- * the delegation model required by the specification. The system class
+ * the delegation model required by the specification. The bootstrap class
* loader will be queried first, then the local repositories, and only then
* delegation to the parent class loader will occur. This allows the web
* application to override any shared class except the classes from J2SE.
@@ -1314,7 +1314,7 @@ public abstract class WebappClassLoaderBase extends
URLClassLoader
return clazz;
}
- // (0.2) Try loading the class with the system class loader, to
prevent
+ // (0.2) Try loading the class with the bootstrap class loader, to
prevent
// the webapp from overriding Java SE classes. This
implements
// SRV.10.7.2
String resourceName = binaryNameToPath(name, false);
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
