You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tomee.apache.org by rm...@apache.org on 2013/02/06 11:40:30 UTC

svn commit: r1442894 - /tomee/tomee/trunk/tomee/tomee-catalina/src/main/java/org/apache/tomee/catalina/SimpleTomEETcpCluster.java

Author: rmannibucau
Date: Wed Feb  6 10:40:29 2013
New Revision: 1442894

URL: http://svn.apache.org/viewvc?rev=1442894&view=rev
Log:
some comments

Modified:
    tomee/tomee/trunk/tomee/tomee-catalina/src/main/java/org/apache/tomee/catalina/SimpleTomEETcpCluster.java

Modified: tomee/tomee/trunk/tomee/tomee-catalina/src/main/java/org/apache/tomee/catalina/SimpleTomEETcpCluster.java
URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/tomee/tomee-catalina/src/main/java/org/apache/tomee/catalina/SimpleTomEETcpCluster.java?rev=1442894&r1=1442893&r2=1442894&view=diff
==============================================================================
--- tomee/tomee/trunk/tomee/tomee-catalina/src/main/java/org/apache/tomee/catalina/SimpleTomEETcpCluster.java (original)
+++ tomee/tomee/trunk/tomee/tomee-catalina/src/main/java/org/apache/tomee/catalina/SimpleTomEETcpCluster.java Wed Feb  6 10:40:29 2013
@@ -62,12 +62,10 @@ public class SimpleTomEETcpCluster exten
 
         // else force the new cluster listener
         for (ClusterListener clusterListener : currentListeners) {
-            clusterListener.setCluster(this);
+            clusterListener.setCluster(this); // we don't care about TomEEClusterListener since it is stateless
         }
 
         super.checkDefaults();
-        addClusterListener(TomEEClusterListener.INSTANCE);
-
-
+        addClusterListener(TomEEClusterListener.INSTANCE); // since that's a singleton and all listeners have to be unique (contains()) we can always add it
     }
 }