This is an automated email from the ASF dual-hosted git repository.

markt pushed a commit to branch 9.0.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git


The following commit(s) were added to refs/heads/9.0.x by this push:
     new b60d60021e Rename some exceptions
b60d60021e is described below

commit b60d60021e384b4f220a030484115df406529c24
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Wed Aug 20 14:34:08 2025 +0100

    Rename some exceptions
    
    Not complete.
---
 java/javax/el/ExpressionFactory.java                   |  6 +++---
 java/org/apache/catalina/startup/HostConfig.java       |  2 +-
 .../catalina/webresources/AbstractFileResourceSet.java |  4 ++--
 java/org/apache/coyote/AbstractProcessor.java          | 18 +++++++++---------
 java/org/apache/coyote/http11/Http11Processor.java     | 18 +++++++++---------
 5 files changed, 24 insertions(+), 24 deletions(-)

diff --git a/java/javax/el/ExpressionFactory.java 
b/java/javax/el/ExpressionFactory.java
index c8fb87b1e6..f2aac0cc88 100644
--- a/java/javax/el/ExpressionFactory.java
+++ b/java/javax/el/ExpressionFactory.java
@@ -359,8 +359,8 @@ public abstract class ExpressionFactory {
                 }
             } catch (FileNotFoundException e) {
                 // Should not happen - ignore it if it does
-            } catch (IOException e) {
-                throw new ELException(Util.message(null, 
"expressionFactory.readFailed", PROPERTY_FILE), e);
+            } catch (IOException ioe) {
+                throw new ELException(Util.message(null, 
"expressionFactory.readFailed", PROPERTY_FILE), ioe);
             }
         }
         return null;
@@ -374,4 +374,4 @@ public abstract class ExpressionFactory {
         return null;
     }
 
-}
+}
\ No newline at end of file
diff --git a/java/org/apache/catalina/startup/HostConfig.java 
b/java/org/apache/catalina/startup/HostConfig.java
index 358ccaf1c7..269b4b22f9 100644
--- a/java/org/apache/catalina/startup/HostConfig.java
+++ b/java/org/apache/catalina/startup/HostConfig.java
@@ -439,7 +439,7 @@ public class HostConfig implements LifecycleListener {
         }
         try {
             return file.getCanonicalFile();
-        } catch (IOException e) {
+        } catch (IOException ioe) {
             return file;
         }
     }
diff --git a/java/org/apache/catalina/webresources/AbstractFileResourceSet.java 
b/java/org/apache/catalina/webresources/AbstractFileResourceSet.java
index fef62fc5f8..54f48da28f 100644
--- a/java/org/apache/catalina/webresources/AbstractFileResourceSet.java
+++ b/java/org/apache/catalina/webresources/AbstractFileResourceSet.java
@@ -106,7 +106,7 @@ public abstract class AbstractFileResourceSet extends 
AbstractResourceSet {
         String canPath = null;
         try {
             canPath = file.getCanonicalPath();
-        } catch (IOException e) {
+        } catch (IOException ignore) {
             // Ignore
         }
         if (canPath == null || !canPath.startsWith(canonicalBase)) {
@@ -256,4 +256,4 @@ public abstract class AbstractFileResourceSet extends 
AbstractResourceSet {
 
 
     protected abstract void checkType(File file);
-}
+}
\ No newline at end of file
diff --git a/java/org/apache/coyote/AbstractProcessor.java 
b/java/org/apache/coyote/AbstractProcessor.java
index 0389c01b36..3b8164eb46 100644
--- a/java/org/apache/coyote/AbstractProcessor.java
+++ b/java/org/apache/coyote/AbstractProcessor.java
@@ -374,8 +374,8 @@ public abstract class AbstractProcessor extends 
AbstractProcessorLight implement
                     try {
                         // Validate and write response headers
                         prepareResponse();
-                    } catch (IOException e) {
-                        handleIOException(e);
+                    } catch (IOException ioe) {
+                        handleIOException(ioe);
                     }
                 }
                 break;
@@ -384,8 +384,8 @@ public abstract class AbstractProcessor extends 
AbstractProcessorLight implement
                 action(ActionCode.COMMIT, null);
                 try {
                     finishResponse();
-                } catch (IOException e) {
-                    handleIOException(e);
+                } catch (IOException ioe) {
+                    handleIOException(ioe);
                 }
                 break;
             }
@@ -396,8 +396,8 @@ public abstract class AbstractProcessor extends 
AbstractProcessorLight implement
             case EARLY_HINTS: {
                 try {
                     earlyHints();
-                } catch (IOException e) {
-                    handleIOException(e);
+                } catch (IOException ioe) {
+                    handleIOException(ioe);
                 }
                 break;
             }
@@ -405,9 +405,9 @@ public abstract class AbstractProcessor extends 
AbstractProcessorLight implement
                 action(ActionCode.COMMIT, null);
                 try {
                     flush();
-                } catch (IOException e) {
-                    handleIOException(e);
-                    response.setErrorException(e);
+                } catch (IOException ioe) {
+                    handleIOException(ioe);
+                    response.setErrorException(ioe);
                 }
                 break;
             }
diff --git a/java/org/apache/coyote/http11/Http11Processor.java 
b/java/org/apache/coyote/http11/Http11Processor.java
index ab0ff058dd..6eb8e36e93 100644
--- a/java/org/apache/coyote/http11/Http11Processor.java
+++ b/java/org/apache/coyote/http11/Http11Processor.java
@@ -301,11 +301,11 @@ public class Http11Processor extends AbstractProcessor {
                         
socketWrapper.setReadTimeout(protocol.getConnectionUploadTimeout());
                     }
                 }
-            } catch (IOException e) {
+            } catch (IOException ioe) {
                 if (log.isDebugEnabled()) {
-                    log.debug(sm.getString("http11processor.header.parse"), e);
+                    log.debug(sm.getString("http11processor.header.parse"), 
ioe);
                 }
-                setErrorState(ErrorState.CLOSE_CONNECTION_NOW, e);
+                setErrorState(ErrorState.CLOSE_CONNECTION_NOW, ioe);
                 break;
             } catch (Throwable t) {
                 ExceptionUtils.handleThrowable(t);
@@ -1200,8 +1200,8 @@ public class Http11Processor extends AbstractProcessor {
         if (getErrorState().isIoAllowed()) {
             try {
                 inputBuffer.endRequest();
-            } catch (IOException e) {
-                setErrorState(ErrorState.CLOSE_CONNECTION_NOW, e);
+            } catch (IOException ioe) {
+                setErrorState(ErrorState.CLOSE_CONNECTION_NOW, ioe);
             } catch (Throwable t) {
                 ExceptionUtils.handleThrowable(t);
                 // 500 - Internal Server Error
@@ -1216,8 +1216,8 @@ public class Http11Processor extends AbstractProcessor {
             try {
                 action(ActionCode.COMMIT, null);
                 outputBuffer.end();
-            } catch (IOException e) {
-                setErrorState(ErrorState.CLOSE_CONNECTION_NOW, e);
+            } catch (IOException ioe) {
+                setErrorState(ErrorState.CLOSE_CONNECTION_NOW, ioe);
             } catch (Throwable t) {
                 ExceptionUtils.handleThrowable(t);
                 setErrorState(ErrorState.CLOSE_NOW, t);
@@ -1251,8 +1251,8 @@ public class Http11Processor extends AbstractProcessor {
             if (!response.isCommitted() && request.hasExpectation()) {
                 try {
                     outputBuffer.sendAck();
-                } catch (IOException e) {
-                    setErrorState(ErrorState.CLOSE_CONNECTION_NOW, e);
+                } catch (IOException ioe) {
+                    setErrorState(ErrorState.CLOSE_CONNECTION_NOW, ioe);
                 }
             }
         }


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to