From 9c0d3ce072142fa0f0a5c032e706178464acbec8 Mon Sep 17 00:00:00 2001
From: James Strachan <jstrachan@apache.org>
Date: Fri, 27 Jan 2006 19:33:21 +0000
Subject: [PATCH] added better debug tracing so that destinations are explicit
 in whether they are a queue or topic

git-svn-id: https://svn.apache.org/repos/asf/incubator/activemq/trunk@372952 13f79535-47bb-0310-9956-ffa450edef68
---
 .../activemq/util/IntrospectionSupport.java   | 32 ++++++++++++++++++-
 .../activemq/command/ActiveMQMessageTest.java |  2 ++
 2 files changed, 33 insertions(+), 1 deletion(-)

diff --git a/activemq-core/src/main/java/org/apache/activemq/util/IntrospectionSupport.java b/activemq-core/src/main/java/org/apache/activemq/util/IntrospectionSupport.java
index 2f3b4026b5..0299df0be1 100755
--- a/activemq-core/src/main/java/org/apache/activemq/util/IntrospectionSupport.java
+++ b/activemq-core/src/main/java/org/apache/activemq/util/IntrospectionSupport.java
@@ -16,6 +16,8 @@
  */
 package org.apache.activemq.util;
 
+import org.apache.activemq.command.ActiveMQDestination;
+
 import java.beans.PropertyEditor;
 import java.beans.PropertyEditorManager;
 import java.lang.reflect.Field;
@@ -28,6 +30,7 @@ import java.util.HashMap;
 import java.util.Iterator;
 import java.util.LinkedHashMap;
 import java.util.Map;
+import java.util.Set;
 import java.util.Map.Entry;
 
 public class IntrospectionSupport {
@@ -149,7 +152,34 @@ public class IntrospectionSupport {
     static public String toString(Object target, Class stopClass) {
         LinkedHashMap map = new LinkedHashMap();
         addFields(target, target.getClass(), stopClass, map);
-        return simpleName(target.getClass())+" "+map;
+        StringBuffer buffer = new StringBuffer(simpleName(target.getClass()));
+        buffer.append(" {");
+        Set entrySet = map.entrySet();
+        boolean first = true;
+        for (Iterator iter = entrySet.iterator(); iter.hasNext();) {
+            Map.Entry entry = (Map.Entry) iter.next();
+            if (first) {
+                first = false;
+            }
+            else {
+                buffer.append(", ");
+            }
+            buffer.append(entry.getKey());
+            buffer.append(" = ");
+            appendToString(buffer, entry.getValue());
+        }
+        buffer.append("}");
+        return buffer.toString();
+    }
+
+    protected static void appendToString(StringBuffer buffer, Object value) {
+        if (value instanceof ActiveMQDestination) {
+            ActiveMQDestination destination = (ActiveMQDestination) value;
+            buffer.append(destination.getQualifiedName());
+        }
+        else {
+            buffer.append(value);
+        }
     }
 
     static public String simpleName(Class clazz) {
diff --git a/activemq-core/src/test/java/org/apache/activemq/command/ActiveMQMessageTest.java b/activemq-core/src/test/java/org/apache/activemq/command/ActiveMQMessageTest.java
index b62834a8c2..f158b80aa7 100755
--- a/activemq-core/src/test/java/org/apache/activemq/command/ActiveMQMessageTest.java
+++ b/activemq-core/src/test/java/org/apache/activemq/command/ActiveMQMessageTest.java
@@ -190,6 +190,8 @@ public class ActiveMQMessageTest extends TestCase {
         assertTrue(msg1.getJMSExpiration() == msg2.getJMSExpiration());
         assertTrue(msg1.getJMSPriority() == msg2.getJMSPriority());
         assertTrue(msg1.getJMSTimestamp() == msg2.getJMSTimestamp());
+        
+        System.out.println("Message is:  " + msg1);
     }
 
     public void testGetAndSetJMSMessageID() throws Exception {