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

commit 1b1f6b82769213c2ba50c4342d8c420cdbf1ede7
Author: Mark Thomas <[email protected]>
AuthorDate: Thu Mar 5 09:03:38 2026 +0000

    Fix Eclipse IDE warnings
---
 test/org/apache/juli/TestFileHandlerNonRotatable.java        | 12 ++++++------
 .../httpd/TestChunkedTransferEncodingWithProxy.java          |  2 +-
 .../tomcat/integration/httpd/TestLargePayloadWithProxy.java  |  4 ++--
 .../tomcat/integration/httpd/TestSessionWithProxy.java       |  4 ++--
 test/org/apache/tomcat/integration/httpd/TesterHttpd.java    |  2 +-
 5 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/test/org/apache/juli/TestFileHandlerNonRotatable.java 
b/test/org/apache/juli/TestFileHandlerNonRotatable.java
index 5b9124544a..376d8ca3ec 100644
--- a/test/org/apache/juli/TestFileHandlerNonRotatable.java
+++ b/test/org/apache/juli/TestFileHandlerNonRotatable.java
@@ -53,28 +53,28 @@ public class TestFileHandlerNonRotatable extends 
LoggingBaseTest {
 
     @Test
     public void testBug61232() throws Exception {
-        testHandler = new FileHandler(this.getTemporaryDirectory().toString(),
+        testHandler = new FileHandler(getTemporaryDirectory().toString(),
                 "juli.", ".log");
 
-        File logFile = new File(this.getTemporaryDirectory(), "juli.log");
+        File logFile = new File(getTemporaryDirectory(), "juli.log");
         Assert.assertTrue(logFile.exists());
     }
 
     @Test
     public void testCustomSuffixWithoutSeparator() throws Exception {
-        testHandler = new FileHandler(this.getTemporaryDirectory().toString(),
+        testHandler = new FileHandler(getTemporaryDirectory().toString(),
                 "juli.", "log");
 
-        File logFile = new File(this.getTemporaryDirectory(), "juli.log");
+        File logFile = new File(getTemporaryDirectory(), "juli.log");
         Assert.assertTrue(logFile.exists());
     }
 
     @Test
     public void testCustomPrefixWithoutSeparator() throws Exception {
-        testHandler = new FileHandler(this.getTemporaryDirectory().toString(),
+        testHandler = new FileHandler(getTemporaryDirectory().toString(),
                 "juli", ".log");
 
-        File logFile = new File(this.getTemporaryDirectory(), "juli.log");
+        File logFile = new File(getTemporaryDirectory(), "juli.log");
         Assert.assertTrue(logFile.exists());
     }
 }
\ No newline at end of file
diff --git 
a/test/org/apache/tomcat/integration/httpd/TestChunkedTransferEncodingWithProxy.java
 
b/test/org/apache/tomcat/integration/httpd/TestChunkedTransferEncodingWithProxy.java
index 4ce39545f4..d69b2e8e4e 100644
--- 
a/test/org/apache/tomcat/integration/httpd/TestChunkedTransferEncodingWithProxy.java
+++ 
b/test/org/apache/tomcat/integration/httpd/TestChunkedTransferEncodingWithProxy.java
@@ -56,7 +56,7 @@ public class TestChunkedTransferEncodingWithProxy extends 
HttpdIntegrationBaseTe
         return HTTPD_CONFIG;
     }
 
-    /**
+    /*
      * Verify that chunked transfer encoding works correctly through the httpd 
reverse proxy
      * which sets proxy-sendchunked to minimize resource usage by using 
chunked encoding.
      */
diff --git 
a/test/org/apache/tomcat/integration/httpd/TestLargePayloadWithProxy.java 
b/test/org/apache/tomcat/integration/httpd/TestLargePayloadWithProxy.java
index 33e4dc35f8..8c404ef54d 100644
--- a/test/org/apache/tomcat/integration/httpd/TestLargePayloadWithProxy.java
+++ b/test/org/apache/tomcat/integration/httpd/TestLargePayloadWithProxy.java
@@ -54,7 +54,7 @@ public class TestLargePayloadWithProxy extends 
HttpdIntegrationBaseTest {
         return HTTPD_CONFIG;
     }
 
-    /**
+    /*
      * Verify that a large POST body passes through the httpd reverse proxy
      */
     @Test
@@ -72,7 +72,7 @@ public class TestLargePayloadWithProxy extends 
HttpdIntegrationBaseTest {
         Assert.assertEquals(String.valueOf(PAYLOAD_SIZE), 
requestDesc.getRequestInfo("REQUEST-BODY-SIZE"));
     }
 
-    /**
+    /*
      * Verify that chunked transfer encoding works correctly through the httpd 
reverse proxy.
      */
     @Test
diff --git a/test/org/apache/tomcat/integration/httpd/TestSessionWithProxy.java 
b/test/org/apache/tomcat/integration/httpd/TestSessionWithProxy.java
index 39e46a8154..3e2a422afb 100644
--- a/test/org/apache/tomcat/integration/httpd/TestSessionWithProxy.java
+++ b/test/org/apache/tomcat/integration/httpd/TestSessionWithProxy.java
@@ -69,7 +69,7 @@ public class TestSessionWithProxy extends 
HttpdIntegrationBaseTest {
         return HTTPD_CONFIG;
     }
 
-    /**
+    /*
      * Verify that a session created through httpd can be retrieved
      * on a subsequent request using the session cookie.
      */
@@ -102,7 +102,7 @@ public class TestSessionWithProxy extends 
HttpdIntegrationBaseTest {
         Assert.assertEquals("false", 
requestDesc.getRequestInfo("SESSION-IS-NEW"));
     }
 
-    /**
+    /*
      * Verify that when SSL is used at httpd, but not Tomcat, and RemoteIpValve
      * sets the scheme to https, session cookies have the Secure flag.
      */
diff --git a/test/org/apache/tomcat/integration/httpd/TesterHttpd.java 
b/test/org/apache/tomcat/integration/httpd/TesterHttpd.java
index 60b0b26e99..95e2f64550 100644
--- a/test/org/apache/tomcat/integration/httpd/TesterHttpd.java
+++ b/test/org/apache/tomcat/integration/httpd/TesterHttpd.java
@@ -145,7 +145,7 @@ public class TesterHttpd {
     private boolean isHttpdReady() throws InterruptedException {
         long deadline = System.currentTimeMillis() + 1000;
         while (System.currentTimeMillis() < deadline) {
-            try (Socket ignored = new Socket("localhost", this.httpdPort)) {
+            try (@SuppressWarnings("unused") Socket ignored = new 
Socket("localhost", this.httpdPort)) {
                 return true;
             } catch (IOException e) {
                 Thread.sleep(100);


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to