You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by fh...@apache.org on 2006/03/21 18:55:21 UTC

svn commit: r387590 - /tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/session/StandardManager.java

Author: fhanik
Date: Tue Mar 21 09:55:07 2006
New Revision: 387590

URL: http://svn.apache.org/viewcvs?rev=387590&view=rev
Log:
In order to make this manager extendable, it should not assume the class of the session

Modified:
    tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/session/StandardManager.java

Modified: tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/session/StandardManager.java
URL: http://svn.apache.org/viewcvs/tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/session/StandardManager.java?rev=387590&r1=387589&r2=387590&view=diff
==============================================================================
--- tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/session/StandardManager.java (original)
+++ tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/session/StandardManager.java Tue Mar 21 09:55:07 2006
@@ -671,7 +671,7 @@
         // Expire all active sessions
         Session sessions[] = findSessions();
         for (int i = 0; i < sessions.length; i++) {
-            StandardSession session = (StandardSession) sessions[i];
+            Session session = sessions[i];
             try {
                 if (session.isValid()) {
                     session.expire();



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


Re: svn commit: r387590 - /tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/session/StandardManager.java

Posted by Filip Hanik - Dev Lists <de...@hanik.com>.
you got it, phase two.

Remy Maucherat wrote:
> Filip Hanik - Dev Lists wrote:
>> everyone kosher with this change?
>
> I like it.
>
> I also remember you agreed a million years ago that the clustering 
> session(s) implementation(s) should extend StandardSession (hehe).
>
> Rémy
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
> For additional commands, e-mail: dev-help@tomcat.apache.org
>


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


Re: svn commit: r387590 - /tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/session/StandardManager.java

Posted by Remy Maucherat <re...@apache.org>.
Filip Hanik - Dev Lists wrote:
> everyone kosher with this change?

I like it.

I also remember you agreed a million years ago that the clustering 
session(s) implementation(s) should extend StandardSession (hehe).

Rémy

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


Re: svn commit: r387590 - /tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/session/StandardManager.java

Posted by Filip Hanik - Dev Lists <de...@hanik.com>.
everyone kosher with this change?

Filip

fhanik@apache.org wrote:
> Author: fhanik
> Date: Tue Mar 21 09:55:07 2006
> New Revision: 387590
>
> URL: http://svn.apache.org/viewcvs?rev=387590&view=rev
> Log:
> In order to make this manager extendable, it should not assume the class of the session
>
> Modified:
>     tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/session/StandardManager.java
>
> Modified: tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/session/StandardManager.java
> URL: http://svn.apache.org/viewcvs/tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/session/StandardManager.java?rev=387590&r1=387589&r2=387590&view=diff
> ==============================================================================
> --- tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/session/StandardManager.java (original)
> +++ tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/session/StandardManager.java Tue Mar 21 09:55:07 2006
> @@ -671,7 +671,7 @@
>          // Expire all active sessions
>          Session sessions[] = findSessions();
>          for (int i = 0; i < sessions.length; i++) {
> -            StandardSession session = (StandardSession) sessions[i];
> +            Session session = sessions[i];
>              try {
>                  if (session.isValid()) {
>                      session.expire();
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
> For additional commands, e-mail: dev-help@tomcat.apache.org
>
>   


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