Author: remm
Date: Mon May 28 03:06:31 2007
New Revision: 542185

URL: http://svn.apache.org/viewvc?view=rev&rev=542185
Log:
- Redo findSessions which was outdated (42530 claims concurrency problems that 
I can't reproduce).

Modified:
    tomcat/trunk/java/org/apache/catalina/session/ManagerBase.java

Modified: tomcat/trunk/java/org/apache/catalina/session/ManagerBase.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/session/ManagerBase.java?view=diff&rev=542185&r1=542184&r2=542185
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/session/ManagerBase.java (original)
+++ tomcat/trunk/java/org/apache/catalina/session/ManagerBase.java Mon May 28 
03:06:31 2007
@@ -175,7 +175,7 @@
      * The set of currently active Sessions for this Manager, keyed by
      * session identifier.
      */
-    protected Map sessions = new ConcurrentHashMap();
+    protected Map<String, Session> sessions = new ConcurrentHashMap<String, 
Session>();
 
     // Number of sessions created by this manager
     protected int sessionCounter=0;
@@ -870,12 +870,7 @@
      */
     public Session[] findSessions() {
 
-        Session results[] = null;
-        synchronized (sessions) {
-            results = new Session[sessions.size()];
-            results = (Session[]) sessions.values().toArray(results);
-        }
-        return (results);
+        return sessions.values().toArray(new Session[0]);
 
     }
 



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to