svn commit: r392624 - /incubator/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/jmx/ManagedRegionBroker.java

[rajdavies]

Author: rajdavies
Date: Sat Apr  8 15:48:33 2006
New Revision: 392624

URL: http://svn.apache.org/viewcvs?rev=392624&view=rev
Log:
fix for http://issues.apache.org/activemq/browse/AMQ-683

Modified:
    
incubator/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/jmx/ManagedRegionBroker.java

Modified: 
incubator/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/jmx/ManagedRegionBroker.java
URL: 
http://svn.apache.org/viewcvs/incubator/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/jmx/ManagedRegionBroker.java?rev=392624&r1=392623&r2=392624&view=diff
==============================================================================
--- 
incubator/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/jmx/ManagedRegionBroker.java
 (original)
+++ 
incubator/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/jmx/ManagedRegionBroker.java
 Sat Apr  8 15:48:33 2006
@@ -186,7 +186,7 @@
     }
 
     public void unregisterSubscription(Subscription sub){
-        ObjectName name=(ObjectName) subscriptionMap.get(sub);
+        ObjectName name=(ObjectName) subscriptionMap.remove(sub);
         if(name!=null){
             try{
                 unregisterSubscription(name);
@@ -331,7 +331,7 @@
             try{
                 c[i]=OpenTypeSupport.convert((Message) messages.get(i));
             }catch(Throwable e){
-                e.printStackTrace();
+                log.error("failed to browse : " + view,e);
             }
         }
         return c;