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
The following commit(s) were added to refs/heads/main by this push:
new 307008faba Code clean-up - no functional change. Use 't' for throwable
307008faba is described below
commit 307008fabac4b8f5c8e4cb988a0775fc880eaaa6
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Tue Aug 19 12:31:06 2025 +0100
Code clean-up - no functional change. Use 't' for throwable
Find and replace with Cursor
Checked manually
---
.../catalina/core/ApplicationDispatcher.java | 16 ++++-----
.../catalina/core/ApplicationFilterChain.java | 12 +++----
java/org/apache/catalina/core/ContainerBase.java | 6 ++--
java/org/apache/catalina/core/StandardWrapper.java | 10 +++---
.../apache/catalina/core/StandardWrapperValve.java | 38 +++++++++++-----------
java/org/apache/catalina/realm/JAASRealm.java | 12 +++----
.../apache/catalina/realm/UserDatabaseRealm.java | 6 ++--
.../catalina/valves/AbstractAccessLogValve.java | 4 +--
.../org/apache/catalina/valves/AccessLogValve.java | 24 +++++++-------
.../catalina/valves/ExtendedAccessLogValve.java | 8 ++---
java/org/apache/coyote/AbstractProtocol.java | 12 +++----
java/org/apache/jasper/servlet/JspServlet.java | 10 +++---
java/org/apache/tomcat/util/net/NioEndpoint.java | 12 +++----
.../apache/tomcat/util/net/SocketWrapperBase.java | 6 ++--
14 files changed, 88 insertions(+), 88 deletions(-)
diff --git a/java/org/apache/catalina/core/ApplicationDispatcher.java
b/java/org/apache/catalina/core/ApplicationDispatcher.java
index 2d3080f0f6..5ae7f01020 100644
--- a/java/org/apache/catalina/core/ApplicationDispatcher.java
+++ b/java/org/apache/catalina/core/ApplicationDispatcher.java
@@ -496,11 +496,11 @@ final class ApplicationDispatcher implements
AsyncDispatcher, RequestDispatcher
wrapper.getLogger().error(sm.getString("applicationDispatcher.allocateException",
wrapper.getName()),
StandardWrapper.getRootCause(e));
servletException = e;
- } catch (Throwable e) {
- ExceptionUtils.handleThrowable(e);
-
wrapper.getLogger().error(sm.getString("applicationDispatcher.allocateException",
wrapper.getName()), e);
+ } catch (Throwable t) {
+ ExceptionUtils.handleThrowable(t);
+
wrapper.getLogger().error(sm.getString("applicationDispatcher.allocateException",
wrapper.getName()), t);
servletException =
- new
ServletException(sm.getString("applicationDispatcher.allocateException",
wrapper.getName()), e);
+ new
ServletException(sm.getString("applicationDispatcher.allocateException",
wrapper.getName()), t);
// servlet = null; is already done so no need to do it explicitly
}
@@ -548,11 +548,11 @@ final class ApplicationDispatcher implements
AsyncDispatcher, RequestDispatcher
} catch (ServletException e) {
wrapper.getLogger().error(sm.getString("applicationDispatcher.deallocateException",
wrapper.getName()), e);
servletException = e;
- } catch (Throwable e) {
- ExceptionUtils.handleThrowable(e);
-
wrapper.getLogger().error(sm.getString("applicationDispatcher.deallocateException",
wrapper.getName()), e);
+ } catch (Throwable t) {
+ ExceptionUtils.handleThrowable(t);
+
wrapper.getLogger().error(sm.getString("applicationDispatcher.deallocateException",
wrapper.getName()), t);
servletException = new ServletException(
- sm.getString("applicationDispatcher.deallocateException",
wrapper.getName()), e);
+ sm.getString("applicationDispatcher.deallocateException",
wrapper.getName()), t);
}
// Reset the old context class loader
diff --git a/java/org/apache/catalina/core/ApplicationFilterChain.java
b/java/org/apache/catalina/core/ApplicationFilterChain.java
index 675aac9d65..b9c04e1cac 100644
--- a/java/org/apache/catalina/core/ApplicationFilterChain.java
+++ b/java/org/apache/catalina/core/ApplicationFilterChain.java
@@ -109,9 +109,9 @@ public final class ApplicationFilterChain implements
FilterChain {
filter.doFilter(request, response, this);
} catch (IOException | ServletException | RuntimeException e) {
throw e;
- } catch (Throwable e) {
- ExceptionUtils.handleThrowable(e);
- throw new ServletException(sm.getString("filterChain.filter"),
e);
+ } catch (Throwable t) {
+ ExceptionUtils.handleThrowable(t);
+ throw new ServletException(sm.getString("filterChain.filter"),
t);
}
return;
}
@@ -130,9 +130,9 @@ public final class ApplicationFilterChain implements
FilterChain {
servlet.service(request, response);
} catch (IOException | ServletException | RuntimeException e) {
throw e;
- } catch (Throwable e) {
- ExceptionUtils.handleThrowable(e);
- throw new ServletException(sm.getString("filterChain.servlet"), e);
+ } catch (Throwable t) {
+ ExceptionUtils.handleThrowable(t);
+ throw new ServletException(sm.getString("filterChain.servlet"), t);
} finally {
if (dispatcherWrapsSameObject) {
lastServicedRequest.set(null);
diff --git a/java/org/apache/catalina/core/ContainerBase.java
b/java/org/apache/catalina/core/ContainerBase.java
index 3eea0ead22..b5f30b7355 100644
--- a/java/org/apache/catalina/core/ContainerBase.java
+++ b/java/org/apache/catalina/core/ContainerBase.java
@@ -721,12 +721,12 @@ public abstract class ContainerBase extends
LifecycleMBeanBase implements Contai
for (Future<Void> result : results) {
try {
result.get();
- } catch (Throwable e) {
- log.error(sm.getString("containerBase.threadedStartFailed"),
e);
+ } catch (Throwable t) {
+ log.error(sm.getString("containerBase.threadedStartFailed"),
t);
if (multiThrowable == null) {
multiThrowable = new MultiThrowable();
}
- multiThrowable.add(e);
+ multiThrowable.add(t);
}
}
diff --git a/java/org/apache/catalina/core/StandardWrapper.java
b/java/org/apache/catalina/core/StandardWrapper.java
index af5ce6243e..32d7e5d799 100644
--- a/java/org/apache/catalina/core/StandardWrapper.java
+++ b/java/org/apache/catalina/core/StandardWrapper.java
@@ -578,9 +578,9 @@ public class StandardWrapper extends ContainerBase
implements ServletConfig, Wra
countAllocated.incrementAndGet();
} catch (ServletException e) {
throw e;
- } catch (Throwable e) {
- ExceptionUtils.handleThrowable(e);
- throw new
ServletException(sm.getString("standardWrapper.allocate"), e);
+ } catch (Throwable t) {
+ ExceptionUtils.handleThrowable(t);
+ throw new
ServletException(sm.getString("standardWrapper.allocate"), t);
}
}
if (!instanceInitialized) {
@@ -758,8 +758,8 @@ public class StandardWrapper extends ContainerBase
implements ServletConfig, Wra
unavailable(null);
// Restore the context ClassLoader
throw new
ServletException(sm.getString("standardWrapper.notServlet", servletClass), e);
- } catch (Throwable e) {
- Throwable throwable =
ExceptionUtils.unwrapInvocationTargetException(e);
+ } catch (Throwable t) {
+ Throwable throwable =
ExceptionUtils.unwrapInvocationTargetException(t);
ExceptionUtils.handleThrowable(throwable);
unavailable(null);
diff --git a/java/org/apache/catalina/core/StandardWrapperValve.java
b/java/org/apache/catalina/core/StandardWrapperValve.java
index 1f5c285384..001442e461 100644
--- a/java/org/apache/catalina/core/StandardWrapperValve.java
+++ b/java/org/apache/catalina/core/StandardWrapperValve.java
@@ -122,11 +122,11 @@ final class StandardWrapperValve extends ValveBase {
StandardWrapper.getRootCause(e));
throwable = e;
exception(request, response, e);
- } catch (Throwable e) {
- ExceptionUtils.handleThrowable(e);
-
container.getLogger().error(sm.getString("standardWrapper.allocateException",
wrapper.getName()), e);
- throwable = e;
- exception(request, response, e);
+ } catch (Throwable t) {
+ ExceptionUtils.handleThrowable(t);
+
container.getLogger().error(sm.getString("standardWrapper.allocateException",
wrapper.getName()), t);
+ throwable = t;
+ exception(request, response, t);
// servlet = null; is set here
}
@@ -210,12 +210,12 @@ final class StandardWrapperValve extends ValveBase {
}
throwable = e;
exception(request, response, e, e.getErrorCode());
- } catch (Throwable e) {
- ExceptionUtils.handleThrowable(e);
+ } catch (Throwable t) {
+ ExceptionUtils.handleThrowable(t);
container.getLogger()
- .error(sm.getString("standardWrapper.serviceException",
wrapper.getName(), context.getName()), e);
- throwable = e;
- exception(request, response, e);
+ .error(sm.getString("standardWrapper.serviceException",
wrapper.getName(), context.getName()), t);
+ throwable = t;
+ exception(request, response, t);
} finally {
// Release the filter chain (if any) for this request
if (filterChain != null) {
@@ -227,12 +227,12 @@ final class StandardWrapperValve extends ValveBase {
if (servlet != null) {
wrapper.deallocate(servlet);
}
- } catch (Throwable e) {
- ExceptionUtils.handleThrowable(e);
-
container.getLogger().error(sm.getString("standardWrapper.deallocateException",
wrapper.getName()), e);
+ } catch (Throwable t) {
+ ExceptionUtils.handleThrowable(t);
+
container.getLogger().error(sm.getString("standardWrapper.deallocateException",
wrapper.getName()), t);
if (throwable == null) {
- throwable = e;
- exception(request, response, e);
+ throwable = t;
+ exception(request, response, t);
}
}
@@ -242,11 +242,11 @@ final class StandardWrapperValve extends ValveBase {
if ((servlet != null) && (wrapper.getAvailable() ==
Long.MAX_VALUE)) {
wrapper.unload();
}
- } catch (Throwable e) {
- ExceptionUtils.handleThrowable(e);
-
container.getLogger().error(sm.getString("standardWrapper.unloadException",
wrapper.getName()), e);
+ } catch (Throwable t) {
+ ExceptionUtils.handleThrowable(t);
+
container.getLogger().error(sm.getString("standardWrapper.unloadException",
wrapper.getName()), t);
if (throwable == null) {
- exception(request, response, e);
+ exception(request, response, t);
}
}
long t2 = System.currentTimeMillis();
diff --git a/java/org/apache/catalina/realm/JAASRealm.java
b/java/org/apache/catalina/realm/JAASRealm.java
index 0fa9a1f20a..1deeb9c6f2 100644
--- a/java/org/apache/catalina/realm/JAASRealm.java
+++ b/java/org/apache/catalina/realm/JAASRealm.java
@@ -360,9 +360,9 @@ public class JAASRealm extends RealmBase {
try {
Configuration config = getConfig();
loginContext = new LoginContext(appName, null,
callbackHandler, config);
- } catch (Throwable e) {
- ExceptionUtils.handleThrowable(e);
- log.error(sm.getString("jaasRealm.unexpectedError"), e);
+ } catch (Throwable t) {
+ ExceptionUtils.handleThrowable(t);
+ log.error(sm.getString("jaasRealm.unexpectedError"), t);
// There is configuration issue with JAAS so mark the realm as
// unavailable
invocationSuccess = false;
@@ -423,9 +423,9 @@ public class JAASRealm extends RealmBase {
// invocations so mark JAAS realm as available
invocationSuccess = true;
return null;
- } catch (Throwable e) {
- ExceptionUtils.handleThrowable(e);
- log.error(sm.getString("jaasRealm.unexpectedError"), e);
+ } catch (Throwable t) {
+ ExceptionUtils.handleThrowable(t);
+ log.error(sm.getString("jaasRealm.unexpectedError"), t);
// JAAS throws exception different from LoginException so mark
the
// realm as unavailable
invocationSuccess = false;
diff --git a/java/org/apache/catalina/realm/UserDatabaseRealm.java
b/java/org/apache/catalina/realm/UserDatabaseRealm.java
index f392ab7355..7893b61f65 100644
--- a/java/org/apache/catalina/realm/UserDatabaseRealm.java
+++ b/java/org/apache/catalina/realm/UserDatabaseRealm.java
@@ -224,10 +224,10 @@ public class UserDatabaseRealm extends RealmBase {
context = getServer().getGlobalNamingContext();
}
database = (UserDatabase) context.lookup(resourceName);
- } catch (Throwable e) {
- ExceptionUtils.handleThrowable(e);
+ } catch (Throwable t) {
+ ExceptionUtils.handleThrowable(t);
if (containerLog != null) {
-
containerLog.error(sm.getString("userDatabaseRealm.lookup", resourceName), e);
+
containerLog.error(sm.getString("userDatabaseRealm.lookup", resourceName), t);
}
database = null;
}
diff --git a/java/org/apache/catalina/valves/AbstractAccessLogValve.java
b/java/org/apache/catalina/valves/AbstractAccessLogValve.java
index c320c90433..0b210f8ef2 100644
--- a/java/org/apache/catalina/valves/AbstractAccessLogValve.java
+++ b/java/org/apache/catalina/valves/AbstractAccessLogValve.java
@@ -789,8 +789,8 @@ public abstract class AbstractAccessLogValve extends
ValveBase implements Access
String init;
try {
init = InetAddress.getLocalHost().getHostAddress();
- } catch (Throwable e) {
- ExceptionUtils.handleThrowable(e);
+ } catch (Throwable t) {
+ ExceptionUtils.handleThrowable(t);
init = "127.0.0.1";
}
diff --git a/java/org/apache/catalina/valves/AccessLogValve.java
b/java/org/apache/catalina/valves/AccessLogValve.java
index b5461edc16..43e6a2633e 100644
--- a/java/org/apache/catalina/valves/AccessLogValve.java
+++ b/java/org/apache/catalina/valves/AccessLogValve.java
@@ -446,9 +446,9 @@ public class AccessLogValve extends AbstractAccessLogValve {
close(false);
try {
holder.renameTo(new File(newFileName));
- } catch (Throwable e) {
- ExceptionUtils.handleThrowable(e);
- log.error(sm.getString("accessLogValve.rotateFail"), e);
+ } catch (Throwable t) {
+ ExceptionUtils.handleThrowable(t);
+ log.error(sm.getString("accessLogValve.rotateFail"), t);
}
/* Make sure date is correct */
@@ -515,9 +515,9 @@ public class AccessLogValve extends AbstractAccessLogValve {
if (!rotatedLogFile.renameTo(newLogFile)) {
log.error(sm.getString("accessLogValve.renameFail",
rotatedLogFile, newLogFile));
}
- } catch (Throwable e) {
- ExceptionUtils.handleThrowable(e);
- log.error(sm.getString("accessLogValve.renameFail",
rotatedLogFile, newLogFile), e);
+ } catch (Throwable t) {
+ ExceptionUtils.handleThrowable(t);
+ log.error(sm.getString("accessLogValve.renameFail",
rotatedLogFile, newLogFile), t);
}
}
}
@@ -541,9 +541,9 @@ public class AccessLogValve extends AbstractAccessLogValve {
if (!currentLogFile.renameTo(newLogFile)) {
log.error(sm.getString("accessLogValve.renameFail",
currentLogFile, newLogFile));
}
- } catch (Throwable e) {
- ExceptionUtils.handleThrowable(e);
- log.error(sm.getString("accessLogValve.renameFail",
currentLogFile, newLogFile), e);
+ } catch (Throwable t) {
+ ExceptionUtils.handleThrowable(t);
+ log.error(sm.getString("accessLogValve.renameFail",
currentLogFile, newLogFile), t);
}
} else {
log.error(sm.getString("accessLogValve.alreadyExists",
currentLogFile, newLogFile));
@@ -566,9 +566,9 @@ public class AccessLogValve extends AbstractAccessLogValve {
if (currentLogFile != null && !currentLogFile.exists()) {
try {
close(false);
- } catch (Throwable e) {
- ExceptionUtils.handleThrowable(e);
- log.info(sm.getString("accessLogValve.closeFail"), e);
+ } catch (Throwable t) {
+ ExceptionUtils.handleThrowable(t);
+ log.info(sm.getString("accessLogValve.closeFail"), t);
}
/* Make sure date is correct */
diff --git a/java/org/apache/catalina/valves/ExtendedAccessLogValve.java
b/java/org/apache/catalina/valves/ExtendedAccessLogValve.java
index 5dec3a3eac..86ed004819 100644
--- a/java/org/apache/catalina/valves/ExtendedAccessLogValve.java
+++ b/java/org/apache/catalina/valves/ExtendedAccessLogValve.java
@@ -108,8 +108,8 @@ public class ExtendedAccessLogValve extends AccessLogValve {
try {
svalue = value.toString();
- } catch (Throwable e) {
- ExceptionUtils.handleThrowable(e);
+ } catch (Throwable t) {
+ ExceptionUtils.handleThrowable(t);
/* Log error */
buf.append('-');
return;
@@ -554,8 +554,8 @@ public class ExtendedAccessLogValve extends AccessLogValve {
String value;
try {
value = InetAddress.getLocalHost().getHostName();
- } catch (Throwable e) {
- ExceptionUtils.handleThrowable(e);
+ } catch (Throwable t) {
+ ExceptionUtils.handleThrowable(t);
value = "localhost";
}
buf.append(value);
diff --git a/java/org/apache/coyote/AbstractProtocol.java
b/java/org/apache/coyote/AbstractProtocol.java
index 1ca099fa39..440eb57e80 100644
--- a/java/org/apache/coyote/AbstractProtocol.java
+++ b/java/org/apache/coyote/AbstractProtocol.java
@@ -1017,9 +1017,9 @@ public abstract class AbstractProtocol<S> implements
ProtocolHandler, MBeanRegis
} finally {
try {
instanceManager.destroyInstance(httpUpgradeHandler);
- } catch (Throwable e) {
- ExceptionUtils.handleThrowable(e);
-
getLog().error(sm.getString("abstractConnectionHandler.error"), e);
+ } catch (Throwable t) {
+ ExceptionUtils.handleThrowable(t);
+
getLog().error(sm.getString("abstractConnectionHandler.error"), t);
}
upgradeToken.contextBind().unbind(oldCL);
}
@@ -1054,12 +1054,12 @@ public abstract class AbstractProtocol<S> implements
ProtocolHandler, MBeanRegis
// Worst case, it isn't recoverable and the attempt at logging
// will trigger another OOME.
getLog().error(sm.getString("abstractConnectionHandler.oome"),
oome);
- } catch (Throwable e) {
- ExceptionUtils.handleThrowable(e);
+ } catch (Throwable t) {
+ ExceptionUtils.handleThrowable(t);
// any other exception or error is odd. Here we log it
// with "ERROR" level, so it will show up even on
// less-than-verbose logs.
-
getLog().error(sm.getString("abstractConnectionHandler.error"), e);
+
getLog().error(sm.getString("abstractConnectionHandler.error"), t);
}
// Make sure socket/processor is removed from the list of current
diff --git a/java/org/apache/jasper/servlet/JspServlet.java
b/java/org/apache/jasper/servlet/JspServlet.java
index d4dad0274d..6643e1213b 100644
--- a/java/org/apache/jasper/servlet/JspServlet.java
+++ b/java/org/apache/jasper/servlet/JspServlet.java
@@ -88,8 +88,8 @@ public class JspServlet extends HttpServlet implements
PeriodicEventListener {
Constructor<?> ctor =
engineOptionsClass.getConstructor(ctorSig);
Object[] args = { config, context };
options = (Options) ctor.newInstance(args);
- } catch (Throwable e) {
- Throwable throwable =
ExceptionUtils.unwrapInvocationTargetException(e);
+ } catch (Throwable t) {
+ Throwable throwable =
ExceptionUtils.unwrapInvocationTargetException(t);
ExceptionUtils.handleThrowable(throwable);
// Need to localize this.
log.warn(Localizer.getMessage("jsp.warning.engineOptionsClass",
engineOptionsName), throwable);
@@ -288,9 +288,9 @@ public class JspServlet extends HttpServlet implements
PeriodicEventListener {
serviceJspFile(request, response, jspUri, precompile);
} catch (RuntimeException | IOException | ServletException e) {
throw e;
- } catch (Throwable e) {
- ExceptionUtils.handleThrowable(e);
- throw new ServletException(e);
+ } catch (Throwable t) {
+ ExceptionUtils.handleThrowable(t);
+ throw new ServletException(t);
}
}
diff --git a/java/org/apache/tomcat/util/net/NioEndpoint.java
b/java/org/apache/tomcat/util/net/NioEndpoint.java
index 62bcae1a30..450fc53cac 100644
--- a/java/org/apache/tomcat/util/net/NioEndpoint.java
+++ b/java/org/apache/tomcat/util/net/NioEndpoint.java
@@ -1259,10 +1259,10 @@ public class NioEndpoint extends
AbstractNetworkChannelEndpoint<NioChannel,Socke
getSocket().free();
}
}
- } catch (Throwable e) {
- ExceptionUtils.handleThrowable(e);
+ } catch (Throwable t) {
+ ExceptionUtils.handleThrowable(t);
if (log.isDebugEnabled()) {
- log.error(sm.getString("endpoint.debug.channelCloseFail"),
e);
+ log.error(sm.getString("endpoint.debug.channelCloseFail"),
t);
}
} finally {
socketBufferHandler = SocketBufferHandler.EMPTY;
@@ -1274,10 +1274,10 @@ public class NioEndpoint extends
AbstractNetworkChannelEndpoint<NioChannel,Socke
if (data != null && data.fchannel != null &&
data.fchannel.isOpen()) {
data.fchannel.close();
}
- } catch (Throwable e) {
- ExceptionUtils.handleThrowable(e);
+ } catch (Throwable t) {
+ ExceptionUtils.handleThrowable(t);
if (log.isDebugEnabled()) {
- log.error(sm.getString("endpoint.sendfile.closeError"), e);
+ log.error(sm.getString("endpoint.sendfile.closeError"), t);
}
}
}
diff --git a/java/org/apache/tomcat/util/net/SocketWrapperBase.java
b/java/org/apache/tomcat/util/net/SocketWrapperBase.java
index d3b134e2b3..1457791521 100644
--- a/java/org/apache/tomcat/util/net/SocketWrapperBase.java
+++ b/java/org/apache/tomcat/util/net/SocketWrapperBase.java
@@ -420,10 +420,10 @@ public abstract class SocketWrapperBase<E> {
if (closed.compareAndSet(false, true)) {
try {
getEndpoint().getHandler().release(this);
- } catch (Throwable e) {
- ExceptionUtils.handleThrowable(e);
+ } catch (Throwable t) {
+ ExceptionUtils.handleThrowable(t);
if (log.isDebugEnabled()) {
- log.error(sm.getString("endpoint.debug.handlerRelease"),
e);
+ log.error(sm.getString("endpoint.debug.handlerRelease"),
t);
}
} finally {
getEndpoint().countDownConnection();
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
