You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by re...@apache.org on 2007/05/28 12:06:33 UTC
svn commit: r542185 -
/tomcat/trunk/java/org/apache/catalina/session/ManagerBase.java
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: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org