From 2373eb2d22db60b179c2a3a63e7c48e93c8aab53 Mon Sep 17 00:00:00 2001
From: Robert Davies <rajdavies@apache.org>
Date: Sat, 8 Apr 2006 22:48:33 +0000
Subject: [PATCH] fix for http://issues.apache.org/activemq/browse/AMQ-683

git-svn-id: https://svn.apache.org/repos/asf/incubator/activemq/trunk@392624 13f79535-47bb-0310-9956-ffa450edef68
---
 .../org/apache/activemq/broker/jmx/ManagedRegionBroker.java   | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/activemq-core/src/main/java/org/apache/activemq/broker/jmx/ManagedRegionBroker.java b/activemq-core/src/main/java/org/apache/activemq/broker/jmx/ManagedRegionBroker.java
index 3b1f08082f..9c0a481ec9 100755
--- a/activemq-core/src/main/java/org/apache/activemq/broker/jmx/ManagedRegionBroker.java
+++ b/activemq-core/src/main/java/org/apache/activemq/broker/jmx/ManagedRegionBroker.java
@@ -186,7 +186,7 @@ public class ManagedRegionBroker extends RegionBroker{
     }
 
     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 @@ public class ManagedRegionBroker extends RegionBroker{
             try{
                 c[i]=OpenTypeSupport.convert((Message) messages.get(i));
             }catch(Throwable e){
-                e.printStackTrace();
+                log.error("failed to browse : " + view,e);
             }
         }
         return c;