You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@activemq.apache.org by ta...@apache.org on 2009/02/27 14:23:41 UTC

svn commit: r748508 - /activemq/activemq-dotnet/Apache.NMS.ActiveMQ/trunk/src/main/csharp/Session.cs

Author: tabish
Date: Fri Feb 27 13:23:40 2009
New Revision: 748508

URL: http://svn.apache.org/viewvc?rev=748508&view=rev
Log:
https://issues.apache.org/activemq/browse/AMQNET-147

Fix to call Connection.removeSession only after the Sessions other resources have been disposed of.

Modified:
    activemq/activemq-dotnet/Apache.NMS.ActiveMQ/trunk/src/main/csharp/Session.cs

Modified: activemq/activemq-dotnet/Apache.NMS.ActiveMQ/trunk/src/main/csharp/Session.cs
URL: http://svn.apache.org/viewvc/activemq/activemq-dotnet/Apache.NMS.ActiveMQ/trunk/src/main/csharp/Session.cs?rev=748508&r1=748507&r2=748508&view=diff
==============================================================================
--- activemq/activemq-dotnet/Apache.NMS.ActiveMQ/trunk/src/main/csharp/Session.cs (original)
+++ activemq/activemq-dotnet/Apache.NMS.ActiveMQ/trunk/src/main/csharp/Session.cs Fri Feb 27 13:23:40 2009
@@ -165,7 +165,6 @@
 				{
 					this.closing = true;
 					StopAsyncDelivery();
-					Connection.RemoveSession(this);
 					lock(consumers.SyncRoot)
 					{
 						foreach(MessageConsumer consumer in consumers.Values)
@@ -183,6 +182,7 @@
 						}
 					}
 					producers.Clear();
+                    Connection.RemoveSession(this);
 				}
 				catch(Exception ex)
 				{