Re: svn commit: r933881 - /tomcat/trunk/java/org/apache/catalina/session/StandardManager.java

[Mladen Turk]

On 04/14/2010 10:48 AM, kfuj...@apache.org wrote:

Author: kfujino

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

+                        session.expire();
+                    }
                      sessionCounter++;

Seems we are using the sessionCounter as well as expiredSessions
just for the information purposes.
Think they should be declared as long instead of int cause
those values never gets reset and can only grow, eventually
reaching the MAX_INT and going to negative values.

Given that Tomcat can survive long enough to
generate +2 billion sessions at the first place :)


Regards
--
^TM

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org