Remove comment - odds of session duplication are so small to be insignificant

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@915075 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/java/org/apache/catalina/session/ManagerBase.java b/java/org/apache/catalina/session/ManagerBase.java
index 865200f..6dd2c49 100644 (file)
--- a/java/org/apache/catalina/session/ManagerBase.java
+++ b/java/org/apache/catalina/session/ManagerBase.java
@@ -821,17 +821,6 @@ public abstract class ManagerBase implements Manager, MBeanRegistration {
         session.setMaxInactiveInterval(this.maxInactiveInterval);
         if (sessionId == null) {
             sessionId = generateSessionId();
-        // FIXME WHy we need no duplication check?
-        /*         
-             synchronized (sessions) {
-                while (sessions.get(sessionId) != null) { // Guarantee
-                    // uniqueness
-                    duplicates++;
-                    sessionId = generateSessionId();
-                }
-            }
-        */
-            
             // FIXME: Code to be used in case route replacement is needed
             /*
         } else {