[jboss-remoting-commits] JBoss Remoting SVN: r4741 - remoting2/branches/2.x/src/main/org/jboss/remoting.

[jboss-remoting-commits at lists.jboss.org]

Author: ron.sigal at jboss.com
Date: 2008-11-28 20:16:55 -0500 (Fri, 28 Nov 2008)
New Revision: 4741

Modified:
   remoting2/branches/2.x/src/main/org/jboss/remoting/Client.java
Log:
JBREM-1069: addConnectionListener(ConnectionListener listener) puts default ping period in metadata map only if ping period is not configured in configuration map.

Modified: remoting2/branches/2.x/src/main/org/jboss/remoting/Client.java
===================================================================
--- remoting2/branches/2.x/src/main/org/jboss/remoting/Client.java	2008-11-29 01:02:41 UTC (rev 4740)
+++ remoting2/branches/2.x/src/main/org/jboss/remoting/Client.java	2008-11-29 01:16:55 UTC (rev 4741)
@@ -376,7 +376,13 @@
     */
    public void addConnectionListener(ConnectionListener listener)
    {
-      addConnectionListener(listener, (int)ConnectionValidator.DEFAULT_PING_PERIOD);
+      HashMap metadata = new HashMap();
+      if (configuration.get(ConnectionValidator.VALIDATOR_PING_PERIOD) == null)
+      {
+         String pingPeriod = Long.toString(ConnectionValidator.DEFAULT_PING_PERIOD);
+         metadata.put(ConnectionValidator.VALIDATOR_PING_PERIOD, pingPeriod);
+      }
+      addConnectionListener(listener, metadata);
    }
 
    /**