You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by jd...@apache.org on 2007/09/18 02:56:58 UTC

svn commit: r576641 - /geronimo/sandbox/gshell/trunk/gshell-remote/gshell-remote-common/src/main/java/org/apache/geronimo/gshell/remote/message/MessageResponseInspector.java

Author: jdillon
Date: Mon Sep 17 17:56:58 2007
New Revision: 576641

URL: http://svn.apache.org/viewvc?rev=576641&view=rev
Log:
Complain when we try to deregister a request that was never registered to start with

Modified:
    geronimo/sandbox/gshell/trunk/gshell-remote/gshell-remote-common/src/main/java/org/apache/geronimo/gshell/remote/message/MessageResponseInspector.java

Modified: geronimo/sandbox/gshell/trunk/gshell-remote/gshell-remote-common/src/main/java/org/apache/geronimo/gshell/remote/message/MessageResponseInspector.java
URL: http://svn.apache.org/viewvc/geronimo/sandbox/gshell/trunk/gshell-remote/gshell-remote-common/src/main/java/org/apache/geronimo/gshell/remote/message/MessageResponseInspector.java?rev=576641&r1=576640&r2=576641&view=diff
==============================================================================
--- geronimo/sandbox/gshell/trunk/gshell-remote/gshell-remote-common/src/main/java/org/apache/geronimo/gshell/remote/message/MessageResponseInspector.java (original)
+++ geronimo/sandbox/gshell/trunk/gshell-remote/gshell-remote-common/src/main/java/org/apache/geronimo/gshell/remote/message/MessageResponseInspector.java Mon Sep 17 17:56:58 2007
@@ -64,6 +64,9 @@
         if (registeredIds.remove(id)) {
             log.debug("Dereegistered request for ID: {}", id);
         }
+        else {
+            log.warn("Ignoring attempt to deregister unregistered request ID: {}", id);
+        }
     }
     
     //