From 2dbecd2538f135239f3d10a0349d51ac16fdc7e7 Mon Sep 17 00:00:00 2001
From: mwebster <mwebster>
Date: Mon, 6 Nov 2006 13:48:34 +0000
Subject: [PATCH]  Bug 155033 "Use ajcore for LTW problems" (call Dump from
 preProcess())

---
 loadtime/src/org/aspectj/weaver/loadtime/Aj.java | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/loadtime/src/org/aspectj/weaver/loadtime/Aj.java b/loadtime/src/org/aspectj/weaver/loadtime/Aj.java
index e4155d833..4df029d2f 100644
--- a/loadtime/src/org/aspectj/weaver/loadtime/Aj.java
+++ b/loadtime/src/org/aspectj/weaver/loadtime/Aj.java
@@ -14,6 +14,7 @@ package org.aspectj.weaver.loadtime;
 import java.util.Map;
 import java.util.WeakHashMap;
 
+import org.aspectj.weaver.Dump;
 import org.aspectj.weaver.tools.Trace;
 import org.aspectj.weaver.tools.TraceFactory;
 import org.aspectj.weaver.tools.WeavingAdaptor;
@@ -75,13 +76,15 @@ public class Aj implements ClassPreProcessor {
                 	return bytes;
                 }
                 byte[] newBytes = weavingAdaptor.weaveClass(className, bytes);
+                Dump.dumpOnExit(weavingAdaptor.getMessageHolder(), true);
         		if (trace.isTraceEnabled()) trace.exit("preProcess",newBytes);
                 return newBytes;
 			}
         
         /* Don't like to do this but JVMTI swallows all exceptions */
         } catch (Throwable th) {
-    		trace.error("preProcess",th);
+    		trace.error(className,th);
+    		Dump.dumpWithException(th);
             //FIXME AV wondering if we should have the option to fail (throw runtime exception) here
             // would make sense at least in test f.e. see TestHelper.handleMessage()
     		if (trace.isTraceEnabled()) trace.exit("preProcess",th);
-- 
2.39.5