You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@activemq.apache.org by ch...@apache.org on 2008/04/18 21:12:11 UTC

svn commit: r649658 - /activemq/trunk/activemq-core/src/main/java/org/apache/activemq/transport/InactivityMonitor.java

Author: chirino
Date: Fri Apr 18 12:12:07 2008
New Revision: 649658

URL: http://svn.apache.org/viewvc?rev=649658&view=rev
Log:
Bad logic was not propagating the transport exception up.

Modified:
    activemq/trunk/activemq-core/src/main/java/org/apache/activemq/transport/InactivityMonitor.java

Modified: activemq/trunk/activemq-core/src/main/java/org/apache/activemq/transport/InactivityMonitor.java
URL: http://svn.apache.org/viewvc/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/transport/InactivityMonitor.java?rev=649658&r1=649657&r2=649658&view=diff
==============================================================================
--- activemq/trunk/activemq-core/src/main/java/org/apache/activemq/transport/InactivityMonitor.java (original)
+++ activemq/trunk/activemq-core/src/main/java/org/apache/activemq/transport/InactivityMonitor.java Fri Apr 18 12:12:07 2008
@@ -239,9 +239,9 @@
     }
 
     public void onException(IOException error) {
-        if (!failed.compareAndSet(false,true)) {
-    		stopMonitorThreads();
-    		transportListener.onException(error);
+        if (failed.compareAndSet(false, true)) {
+            stopMonitorThreads();
+            transportListener.onException(error);
         }
     }