finishResolveTypes now tests for CONCRETE as well

as RESOLVED pointcuts

diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/PointcutDesignator.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/PointcutDesignator.java
index 65c362167d94fc07e4c1e6f53ede3d7a1a9f386c..f741bff7b3d730f44c40eb0fcb0ddc40cd76d7b5 100644 (file)
--- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/PointcutDesignator.java
+++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/PointcutDesignator.java
@@ -61,7 +61,10 @@ public class PointcutDesignator extends AstNode {
                //System.err.println("resolving: " + this);
                //Thread.currentThread().dumpStack();
                //XXX why do we need this test
-               if (pointcut.state == Pointcut.RESOLVED) return true;
+               // AMC added concrete too. Needed because declare declarations concretize their
+               // shadow mungers early.
+               if (pointcut.state == Pointcut.RESOLVED ||
+                       pointcut.state == Pointcut.CONCRETE) return true;
                
                TypeBinding[] parameters = method.parameters;
                Argument[] arguments = dec.arguments;