Author: rjung Date: Thu Nov 10 23:44:18 2011 New Revision: 1200627 URL: http://svn.apache.org/viewvc?rev=1200627&view=rev Log: Use setters when cloning cluster managers (at least in ClusterManagerBase).
Be consistent and don't be afraid of property change listeners. The digester calls the setters as well when initializing the objects. Modified: tomcat/trunk/java/org/apache/catalina/ha/session/ClusterManagerBase.java Modified: tomcat/trunk/java/org/apache/catalina/ha/session/ClusterManagerBase.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/ha/session/ClusterManagerBase.java?rev=1200627&r1=1200626&r2=1200627&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/ha/session/ClusterManagerBase.java (original) +++ tomcat/trunk/java/org/apache/catalina/ha/session/ClusterManagerBase.java Thu Nov 10 23:44:18 2011 @@ -183,7 +183,7 @@ public abstract class ClusterManagerBase protected void clone(ClusterManagerBase copy) { copy.setName("Clone-from-" + getName()); copy.setCluster(getCluster()); - copy.maxActiveSessions = getMaxActiveSessions(); + copy.setMaxActiveSessions(getMaxActiveSessions()); copy.setMaxInactiveInterval(getMaxInactiveInterval()); copy.setSessionIdLength(getSessionIdLength()); copy.setProcessExpiresFrequency(getProcessExpiresFrequency()); --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org