You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2024/11/20 20:21:30 UTC
(tomcat) 01/02: Avoid throw - catch - re-throw for CNFE
This is an automated email from the ASF dual-hosted git repository.
markt pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tomcat.git
commit 614a05290ef4672755029288aa9151bbdaee6f30
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Wed Nov 20 20:15:44 2024 +0000
Avoid throw - catch - re-throw for CNFE
---
java/org/apache/catalina/loader/WebappClassLoaderBase.java | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/java/org/apache/catalina/loader/WebappClassLoaderBase.java b/java/org/apache/catalina/loader/WebappClassLoaderBase.java
index d224e473f8..86f705fd9a 100644
--- a/java/org/apache/catalina/loader/WebappClassLoaderBase.java
+++ b/java/org/apache/catalina/loader/WebappClassLoaderBase.java
@@ -657,18 +657,18 @@ public abstract class WebappClassLoaderBase extends URLClassLoader
throw e;
}
}
- if (clazz == null) {
- if (log.isTraceEnabled()) {
- log.trace(" --> Returning ClassNotFoundException");
- }
- throw new ClassNotFoundException(name);
- }
} catch (ClassNotFoundException e) {
if (log.isTraceEnabled()) {
log.trace(" --> Passing on ClassNotFoundException");
}
throw e;
}
+ if (clazz == null) {
+ if (log.isTraceEnabled()) {
+ log.trace(" --> Returning ClassNotFoundException");
+ }
+ throw new ClassNotFoundException(name);
+ }
// Return the class we have located
if (log.isTraceEnabled()) {
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org