proper fix for 121385.

7 files changed, 133 insertions, 0 deletions

diff --git a/tests/bugs150/pr121385/A.java b/tests/bugs150/pr121385/A.java
new file mode 100644
index 000000000..fd94e0a30
--- /dev/null
+++ b/tests/bugs150/pr121385/A.java
@@ -0,0 +1,16 @@
+import org.aspectj.lang.annotation.*;
+
+ abstract aspect X {
+  void around(): execution(* foo(..)) {}
+}
+
+
+@Aspect class B extends X { }
+
+public class A {
+  public void foo() { } 
+
+public static void main(String []argv) {
+   new A().foo();
+}
+}
diff --git a/tests/bugs150/pr121385/ConcreteWorld.aj b/tests/bugs150/pr121385/ConcreteWorld.aj
new file mode 100644
index 000000000..cb1716509
--- /dev/null
+++ b/tests/bugs150/pr121385/ConcreteWorld.aj
@@ -0,0 +1,5 @@
+public aspect ConcreteWorld extends World {
+    pointcut greeting() :
+    	execution(* Hello.sayWorld(..))
+    	|| execution(* Hello.sayHello(..));
+}
diff --git a/tests/bugs150/pr121385/Hello.java b/tests/bugs150/pr121385/Hello.java
new file mode 100644
index 000000000..833ee1d3e
--- /dev/null
+++ b/tests/bugs150/pr121385/Hello.java
@@ -0,0 +1,16 @@
+public class Hello {
+
+        public static void main(String[] args) {
+                sayHello();
+        } 
+
+        public static void sayHello() {
+                System.out.println("Hello");
+                sayWorld();
+        } 
+
+        public static int sayWorld() {
+                System.out.println("World");
+                return 0;
+        }
+}
diff --git a/tests/bugs150/pr121385/World.aj b/tests/bugs150/pr121385/World.aj
new file mode 100644
index 000000000..ac3fd5cc6
--- /dev/null
+++ b/tests/bugs150/pr121385/World.aj
@@ -0,0 +1,24 @@
+import org.aspectj.lang.Signature;
+import org.aspectj.lang.JoinPoint;
+public abstract aspect World { 
+    //private Object result;
+        pointcut greeting() : execution(* Hello.sayWorld(..)); 
+
+        Object around(): greeting() {
+        System.out.println("around start!");
+        Object result = proceed();
+        System.out.println("around end!");
+        return result;
+        }
+
+//    before() : greeting() { 
+//      Signature signature = thisJoinPoint.getSignature();
+//        System.out.println("before " + signature.getName()); 
+//    } 
+
+//    after() returning () : greeting() { 
+//      Signature signature = thisJoinPoint.getSignature();
+//        System.out.println("after " + signature.getName()); 
+//    } 
+
+} 
diff --git a/tests/bugs150/pr121385/WorldAt.java b/tests/bugs150/pr121385/WorldAt.java
new file mode 100644
index 000000000..1d142de59
--- /dev/null
+++ b/tests/bugs150/pr121385/WorldAt.java
@@ -0,0 +1,10 @@
+import org.aspectj.lang.annotation.Aspect;
+import org.aspectj.lang.annotation.Pointcut;
+
+@Aspect
+public abstract class WorldAt {
+
+	@Pointcut("execution(* Hello.sayWorld(..))")
+    void greeting() {}
+
+}
diff --git a/tests/bugs150/pr121385/ant.xml b/tests/bugs150/pr121385/ant.xml
new file mode 100644
index 000000000..994778ceb
--- /dev/null
+++ b/tests/bugs150/pr121385/ant.xml
@@ -0,0 +1,43 @@
+<!-- ajc-ant script, not to be used from Ant commant line - see AntSpec -->
+<project name="pr120473">
+
+    <!-- using this we can debug the forked VM -->
+    <property
+        name="jdwp"
+        value="-Xdebug -Xnoagent -Djava.compiler=NONE -Xrunjdwp:transport=dt_socket,server=y,suspend=y,address=5005"/>
+
+    <target name="compile:javac">
+        <!-- compile only javac compilable stuff, exclude the one that needs other dependencies -->
+        <javac destdir="${aj.sandbox}" classpathref="aj.path"
+            srcdir="${basedir}"
+            includes="ataspectj/*"
+            excludes="ataspectj/UnweavableTest.java"
+            debug="true">
+        </javac>
+    </target>
+
+    <target name="ltw">
+        <java fork="yes" classname="ataspectj.AllLTWTests" failonerror="yes">
+            <classpath refid="aj.path"/>
+            <!-- use META-INF/aop.xml style -->
+            <classpath path="ataspectj/pathentry"/>
+            <jvmarg value="-javaagent:${aj.root}/lib/test/loadtime5.jar"/>
+<!--            <jvmarg line="${jdwp}"/>-->
+        </java>
+    </target>
+
+    <target name="ConcreteAsepectTest">
+    	<copy file="aop.xml" todir="${aj.sandbox}/META-INF">
+    	</copy>
+        <java fork="yes" classname="Hello" failonerror="yes">
+            <classpath refid="aj.path"/>
+            <jvmarg value="-javaagent:${aj.root}/lib/test/loadtime5.jar"/>
+<!--        	
+            <jvmarg value="-Daj5.def=aop.xml"/>
+            <jvmarg value="-Daj.weaving.verbose=true"/>
+-->            
+<!--            <jvmarg line="${jdwp}"/>            -->
+        </java>
+    </target>
+
+</project>
diff --git a/tests/bugs150/pr121385/aop.xml b/tests/bugs150/pr121385/aop.xml
new file mode 100644
index 000000000..b5e25899e
--- /dev/null
+++ b/tests/bugs150/pr121385/aop.xml
@@ -0,0 +1,19 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<aspectj>
+	<aspects>
+        <concrete-aspect name="World1" extends="World">
+        	<pointcut name="greeting" expression="execution(* Hello.sayWorld(..)) || execution(* Hello.sayHello(..))"/>
+        </concrete-aspect>
+	</aspects>
+<!--
+	<aspects>
+    	<aspect name="World"/>
+    	<aspect name="ConcreteWorld"/>
+	</aspects>
+-->
+
+	<weaver options="-Xreweavable -verbose -XlazyTjp -showWeaveInfo">
+    	<include within="Hello"/>
+	</weaver>
+</aspectj>
+