[jboss-remoting-commits] JBoss Remoting SVN: r5132 - remoting2/branches/2.2.2-SP11_JBREM-1112/src/main/org/jboss/remoting.

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

Author: ron.sigal at jboss.com
Date: 2009-05-05 04:23:32 -0400 (Tue, 05 May 2009)
New Revision: 5132

Modified:
   remoting2/branches/2.2.2-SP11_JBREM-1112/src/main/org/jboss/remoting/LeasePinger.java
Log:
JBREM-1112 (and others to be named): Added some logging.

Modified: remoting2/branches/2.2.2-SP11_JBREM-1112/src/main/org/jboss/remoting/LeasePinger.java
===================================================================
--- remoting2/branches/2.2.2-SP11_JBREM-1112/src/main/org/jboss/remoting/LeasePinger.java	2009-05-05 08:22:57 UTC (rev 5131)
+++ remoting2/branches/2.2.2-SP11_JBREM-1112/src/main/org/jboss/remoting/LeasePinger.java	2009-05-05 08:23:32 UTC (rev 5132)
@@ -70,6 +70,7 @@
 
    public void stopPing()
    {
+       log.debug(this + " entered stopPing()");
       if(trace) { log.trace(this + " stopping lease timer"); }
 
       if (timerTask != null)
@@ -77,10 +78,10 @@
          timerTask.cancel();
          timerTask = null;
          
-         Iterator it = clients.keySet().iterator();
+         Iterator it = clients.values().iterator();
          {
             Client client = (Client) it.next();
-            client.disconnectAndNotify();
+            client.notifyAndDisconnect();
             it.remove();
          }
          
@@ -165,6 +166,10 @@
             startPing();
          }
       }
+      else
+      {
+          log.debug(this + " has already shut down");
+      }
    }
 
    public boolean removeClient(String sessionID)