matthews patches from 116899

diff --git a/loadtime/src/org/aspectj/weaver/loadtime/Options.java b/loadtime/src/org/aspectj/weaver/loadtime/Options.java
index d96f45cb134850f83194c1724d165269eb3ffd15..2cad59e759b6cc8a2d1d394bb34c4434317a2eb6 100644 (file)
--- a/loadtime/src/org/aspectj/weaver/loadtime/Options.java
+++ b/loadtime/src/org/aspectj/weaver/loadtime/Options.java
@@ -128,7 +128,7 @@ public class Options {
             weaverOption.messageHandler.dontIgnore(IMessage.WARNING);
         }
         if (weaverOption.verbose) {
-            weaverOption.messageHandler.dontIgnore(IMessage.DEBUG);
+            weaverOption.messageHandler.dontIgnore(IMessage.INFO);
         }
         if (weaverOption.showWeaveInfo) {
             weaverOption.messageHandler.dontIgnore(IMessage.WEAVEINFO);

diff --git a/tests/java5/ataspectj/ataspectj/ltwlog/Main.java b/tests/java5/ataspectj/ataspectj/ltwlog/Main.java
new file mode 100644 (file)
index 0000000..7bbffd7
--- /dev/null
+++ b/tests/java5/ataspectj/ataspectj/ltwlog/Main.java
@@ -0,0 +1,29 @@
+/*******************************************************************************
+ * Copyright (c) 2005 Contributors.
+ * All rights reserved. 
+ * This program and the accompanying materials are made available 
+ * under the terms of the Eclipse Public License v1.0 
+ * which accompanies this distribution and is available at 
+ * http://eclipse.org/legal/epl-v10.html 
+ * 
+ * Contributors:
+ *   Alexandre Vasseur         initial implementation
+ *******************************************************************************/
+package ataspectj.ltwlog;
+
+import java.util.ArrayList;
+import java.util.Arrays;
+
+/**
+ * @author <a href="mailto:alex AT gnilux DOT com">Alexandre Vasseur</a>
+ */
+public class Main {
+
+    void target() {};
+
+    public static void main(String args[]) throws Throwable {
+        new Main().target();
+    }
+
+
+}

diff --git a/tests/java5/ataspectj/ataspectj/ltwlog/MessageHolder.java b/tests/java5/ataspectj/ataspectj/ltwlog/MessageHolder.java
index 4f245ab358003b28672febf3b8e9992a07625540..03983cfc4a1c06f1beccfb3646fde42f0f9dbde5 100644 (file)
--- a/tests/java5/ataspectj/ataspectj/ltwlog/MessageHolder.java
+++ b/tests/java5/ataspectj/ataspectj/ltwlog/MessageHolder.java
@@ -28,6 +28,7 @@ public class MessageHolder extends DefaultMessageHandler {
     static List s_messages = new ArrayList();
 
     public MessageHolder() {
+       System.out.println("MessageHolder.MessageHolder()");
     }
 
     public boolean handleMessage(IMessage message) throws AbortException {

diff --git a/tests/java5/ataspectj/ataspectj/ltwlog/aop-messagehandler.xml b/tests/java5/ataspectj/ataspectj/ltwlog/aop-messagehandler.xml
new file mode 100644 (file)
index 0000000..16ca3d7
--- /dev/null
+++ b/tests/java5/ataspectj/ataspectj/ltwlog/aop-messagehandler.xml
@@ -0,0 +1,6 @@
+<aspectj>
+    <weaver options="-XmessageHandlerClass:ataspectj.ltwlog.MessageHolder -showWeaveInfo -verbose"/>
+    <aspects>
+        <aspect name="ataspectj.ltwlog.Aspect1"/>
+    </aspects>
+</aspectj>

diff --git a/tests/java5/ataspectj/ataspectj/ltwlog/aop-silent.xml b/tests/java5/ataspectj/ataspectj/ltwlog/aop-silent.xml
index 6e320f1d28d60face3f5602cfa1a80caadde1e0d..0b0a8622f5f854a090d8aee3b679557a821dff66 100644 (file)
--- a/tests/java5/ataspectj/ataspectj/ltwlog/aop-silent.xml
+++ b/tests/java5/ataspectj/ataspectj/ltwlog/aop-silent.xml
@@ -1,5 +1,4 @@
 <aspectj>
-    <weaver options="-XmessageHandlerClass:ataspectj.ltwlog.MessageHolder"/>
     <aspects>
         <aspect name="ataspectj.ltwlog.Aspect1"/>
     </aspects>

diff --git a/tests/java5/ataspectj/ataspectj/ltwlog/aop-verbose.xml b/tests/java5/ataspectj/ataspectj/ltwlog/aop-verbose.xml
index c7756c8e57710347da6e08f269c9160b0930fc97..7cd1b4629d299617e2f61bb71a6a50981ddcc191 100644 (file)
--- a/tests/java5/ataspectj/ataspectj/ltwlog/aop-verbose.xml
+++ b/tests/java5/ataspectj/ataspectj/ltwlog/aop-verbose.xml
@@ -1,5 +1,5 @@
 <aspectj>
-    <weaver options="-XmessageHandlerClass:ataspectj.ltwlog.MessageHolder -verbose"/>
+    <weaver options="-verbose"/>
     <aspects>
         <aspect name="ataspectj.ltwlog.Aspect1"/>
     </aspects>

diff --git a/tests/java5/ataspectj/ataspectj/ltwlog/aop-verboseandshow.xml b/tests/java5/ataspectj/ataspectj/ltwlog/aop-verboseandshow.xml
index 492d6df63df150b93493f757c912f5a5ab730221..557ad27a502b735dc7fc22a5aba029e946a7c987 100644 (file)
--- a/tests/java5/ataspectj/ataspectj/ltwlog/aop-verboseandshow.xml
+++ b/tests/java5/ataspectj/ataspectj/ltwlog/aop-verboseandshow.xml
@@ -1,5 +1,5 @@
 <aspectj>
-    <weaver options="-XmessageHandlerClass:ataspectj.ltwlog.MessageHolder -showWeaveInfo -verbose -Xreweavable"/>
+    <weaver options="-showWeaveInfo -verbose"/>
     <aspects>
         <aspect name="ataspectj.ltwlog.Aspect1"/>
     </aspects>

diff --git a/tests/ltw/aop-abstractaspect.xml b/tests/ltw/aop-abstractaspect.xml
index abd54cc96f3c1b54e6aea277b4821708f69921ea..7f82c94eea4cf66c3bc2f95b2ea733e7638875be 100644 (file)
--- a/tests/ltw/aop-abstractaspect.xml
+++ b/tests/ltw/aop-abstractaspect.xml
@@ -3,6 +3,6 @@
            <aspect name="AbstractAspect"/>
        </aspects>
     
-    <weaver options="-showWeaveInfo -verbose"/>
+    <weaver options="-showWeaveInfo"/>
 </aspectj>
        

diff --git a/tests/src/org/aspectj/systemtest/ajc150/ataspectj/AtAjLTWTests.java b/tests/src/org/aspectj/systemtest/ajc150/ataspectj/AtAjLTWTests.java
index 5b0d24e705cd9383726a3d20a4a3b0c0280ec02a..27d485d4625028f9948e48933452ac6175872cd8 100644 (file)
--- a/tests/src/org/aspectj/systemtest/ajc150/ataspectj/AtAjLTWTests.java
+++ b/tests/src/org/aspectj/systemtest/ajc150/ataspectj/AtAjLTWTests.java
@@ -73,8 +73,20 @@ public class AtAjLTWTests extends XMLBasedAjcTestCase {
         runTest("Ajc Aspect1 LTW Aspect2 -Xreweavable");
     }
 
-    public void testLTWLog() {
-        runTest("LTW Log");
+    public void testLTWLogSilent() {
+        runTest("LTW Log silent");
+    }
+
+    public void testLTWLogVerbose() {
+        runTest("LTW Log verbose");
+    }
+
+    public void testLTWLogVerboseAndShow() {
+        runTest("LTW Log verbose and showWeaveInfo");
+    }
+
+    public void testLTWLogMessageHandlerClass() {
+        runTest("LTW Log messageHandlerClass");
     }
 
     public void testLTWUnweavable() {

diff --git a/tests/src/org/aspectj/systemtest/ajc150/ataspectj/ltw.xml b/tests/src/org/aspectj/systemtest/ajc150/ataspectj/ltw.xml
index 7122a2a709ea34e12d50d7dc542914aebba563dc..c3d0045d83fd2a9d3332dd6565f6f7a65a6f5ebd 100644 (file)
--- a/tests/src/org/aspectj/systemtest/ajc150/ataspectj/ltw.xml
+++ b/tests/src/org/aspectj/systemtest/ajc150/ataspectj/ltw.xml
@@ -94,8 +94,88 @@
         <ant file="ajc-ant.xml" target="ltw.Aspect2MainTest" verbose="true"/>
     </ajc-test>
 
-    <ajc-test dir="java5/ataspectj" title="LTW Log">
-        <ant file="ajc-ant.xml" target="ltwlog.LTWLog" verbose="true"/>
+    <ajc-test dir="java5/ataspectj" title="LTW Log silent">
+       <compile
+               files="ataspectj/ltwlog/Main.java"
+               options="-1.5"
+       />
+       <compile
+               files="ataspectj/ltwlog/Aspect1.java"
+               options="-1.5 -XnoWeave"
+       >
+        </compile>     
+        <run class="ataspectj.ltwlog.Main" ltw="ataspectj/ltwlog/aop-silent.xml">
+               <stdout>
+                               <line text="execution(Main.target())"/>
+            </stdout>
+               <stderr>
+            </stderr>
+        </run>
+    </ajc-test>
+
+    <ajc-test dir="java5/ataspectj" title="LTW Log verbose">
+       <compile
+               files="ataspectj/ltwlog/Main.java"
+               options="-1.5"
+       />
+       <compile
+               files="ataspectj/ltwlog/Aspect1.java"
+               options="-1.5 -XnoWeave"
+       >
+        </compile>     
+        <run class="ataspectj.ltwlog.Main" ltw="ataspectj/ltwlog/aop-verbose.xml">
+               <stdout>
+                               <line text="execution(Main.target())"/>
+            </stdout>
+               <stderr>
+                               <line text="info register aspect ataspectj.ltwlog.Aspect1"/>
+                               <line text="info weaving 'ataspectj.ltwlog.Main'"/>
+                               <line text="info weaving 'ataspectj.ltwlog.Aspect1'"/>
+            </stderr>
+        </run>
+    </ajc-test>
+
+    <ajc-test dir="java5/ataspectj" title="LTW Log verbose and showWeaveInfo">
+       <compile
+               files="ataspectj/ltwlog/Main.java"
+               options="-1.5"
+       />
+       <compile
+               files="ataspectj/ltwlog/Aspect1.java"
+               options="-1.5 -XnoWeave"
+       >
+        </compile>     
+        <run class="ataspectj.ltwlog.Main" ltw="ataspectj/ltwlog/aop-verboseandshow.xml">
+               <stdout>
+                               <line text="execution(Main.target())"/>
+            </stdout>
+               <stderr>
+                               <line text="info register aspect ataspectj.ltwlog.Aspect1"/>
+                               <line text="info weaving 'ataspectj.ltwlog.Main'"/>
+                               <line text="weaveinfo Join point 'method-execution(void ataspectj.ltwlog.Main.target())' in Type 'ataspectj.ltwlog.Main' (Main.java:22) advised by before advice from 'ataspectj.ltwlog.Aspect1' (Aspect1.java)"/>
+                               <line text="info weaving 'ataspectj.ltwlog.Aspect1'"/>
+            </stderr>
+        </run>
+    </ajc-test>
+
+    <ajc-test dir="java5/ataspectj" title="LTW Log messageHandlerClass">
+       <compile
+               files="ataspectj/ltwlog/Main.java, ataspectj/ltwlog/MessageHolder.java"
+               options="-1.5"
+       />
+       <compile
+               files="ataspectj/ltwlog/Aspect1.java"
+               options="-1.5 -XnoWeave"
+       >
+        </compile>     
+        <run class="ataspectj.ltwlog.Main" ltw="ataspectj/ltwlog/aop-messagehandler.xml">
+               <stdout>
+                               <line text="MessageHolder.MessageHolder()"/>
+                               <line text="execution(Main.target())"/>
+            </stdout>
+               <stderr>
+            </stderr>
+        </run>
     </ajc-test>
 
     <ajc-test dir="java5/ataspectj" title="LTW Unweavable">