[jboss-cvs] JBoss Messaging SVN: r3713 - trunk/src/main/org/jboss/jms/client/impl.

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

Author: jmesnil
Date: 2008-02-13 07:45:02 -0500 (Wed, 13 Feb 2008)
New Revision: 3713

Modified:
   trunk/src/main/org/jboss/jms/client/impl/ClientSessionImpl.java
Log:
* added a call to Thread.interrupted() when closing the session to reset the current thread interrupt status (otherwise JBMESSAGING-542 failure occurs in MiscelleaneousTest when the invm optimization is disabled)

Modified: trunk/src/main/org/jboss/jms/client/impl/ClientSessionImpl.java
===================================================================
--- trunk/src/main/org/jboss/jms/client/impl/ClientSessionImpl.java	2008-02-12 21:17:08 UTC (rev 3712)
+++ trunk/src/main/org/jboss/jms/client/impl/ClientSessionImpl.java	2008-02-13 12:45:02 UTC (rev 3713)
@@ -360,6 +360,10 @@
          remotingConnection.send(id, new CloseMessage());
    
          executor.shutdownNow();
+         // executor.shutdownNow() makes a best-effort to cancel threads using Thread.interrupt().
+         // we call Thread.interrupted() to reset the thread status.
+         // without this call, JBMESSAGING-542 happens when closing a connection from a message listener.
+         Thread.interrupted();
       }
       finally
       {