[tomcat] branch master updated: Add comment to explain the cleanup

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view

[tomcat] branch master updated: Add comment to explain the cleanup

Rémy Maucherat
This is an automated email from the ASF dual-hosted git repository.

remm pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/tomcat.git

The following commit(s) were added to refs/heads/master by this push:
     new 67909e0  Add comment to explain the cleanup
67909e0 is described below

commit 67909e03e2a8c00a5fe6b8214f26305a67548fd1
Author: remm <[hidden email]>
AuthorDate: Fri Nov 8 09:38:14 2019 +0100

    Add comment to explain the cleanup
 java/org/apache/tomcat/util/net/AprEndpoint.java | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/java/org/apache/tomcat/util/net/AprEndpoint.java b/java/org/apache/tomcat/util/net/AprEndpoint.java
index 0315075..1d40f29 100644
--- a/java/org/apache/tomcat/util/net/AprEndpoint.java
+++ b/java/org/apache/tomcat/util/net/AprEndpoint.java
@@ -695,6 +695,8 @@ public class AprEndpoint extends AbstractEndpoint<Long,Long> implements SNICallB
             // the pool and its queue are full
             log.error(sm.getString("endpoint.process.fail"), t);
+        // Note: doing connections.remove is not needed since closeSocket will
+        // be called, which then calls close on the wrapper
         return false;

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