AjLookupEnvironment: move phase CONNECT_TYPE_HIERARCHY2 to later

Actually, this is just a guess. I do not understand enough about the
whole lookup process to decide confidently, where the call to
CompilationUnitScope.connectTypeHierarchy2() really should be made.

Splitting the phases in CompilationAndWeavingContext, mimicking what JDT
Core does in LookupEnvironment, is one thing. But deciding when to
initiate the second phase for connecting the type hierarchy, is another
one. Moving it to after BUILD_FIELDS_AND_METHODS is inconsistent with
JDT Core, but at least it fixes the previously failing tests.

Signed-off-by: Alexander Kriegisch <[email protected]>

Author: kriegaex

bridge/src/main/java/org/aspectj/bridge/context/CompilationAndWeavingContext.java

@@ -41,7 +41,7 @@ public class CompilationAndWeavingContext {
 	public static final int COMPLETING_TYPE_BINDINGS = 6;
 	public static final int PROCESSING_DECLARE_PARENTS = 7;
 	public static final int CHECK_AND_SET_IMPORTS = 8;
-	public static final int CONNECTING_TYPE_HIERARCHY = 9;
+	public static final int CONNECTING_TYPE_HIERARCHY1 = 9;
 	public static final int BUILDING_FIELDS_AND_METHODS = 10;
 	public static final int COLLECTING_ITDS_AND_DECLARES = 11;
 	public static final int PROCESSING_DECLARE_ANNOTATIONS = 12;
@@ -54,31 +54,32 @@ public class CompilationAndWeavingContext {
 	public static final int FIXING_SUPER_CALLS_IN_ITDS = 19;
 	public static final int FIXING_SUPER_CALLS = 20;
 	public static final int OPTIMIZING_THIS_JOIN_POINT_CALLS = 21;
+	public static final int CONNECTING_TYPE_HIERARCHY2 = 22;
 
 	// "BACK END"
 
-	public static final int WEAVING = 22;
-	public static final int PROCESSING_REWEAVABLE_STATE = 23;
-	public static final int PROCESSING_TYPE_MUNGERS = 24;
-	public static final int WEAVING_ASPECTS = 25;
-	public static final int WEAVING_CLASSES = 26;
-	public static final int WEAVING_TYPE = 27;
-	public static final int MATCHING_SHADOW = 28;
-	public static final int IMPLEMENTING_ON_SHADOW = 29;
-	public static final int MATCHING_POINTCUT = 30;
-	public static final int MUNGING_WITH = 31;
-	public static final int PROCESSING_ATASPECTJTYPE_MUNGERS_ONLY = 32;
+	public static final int WEAVING = 23;
+	public static final int PROCESSING_REWEAVABLE_STATE = 24;
+	public static final int PROCESSING_TYPE_MUNGERS = 25;
+	public static final int WEAVING_ASPECTS = 26;
+	public static final int WEAVING_CLASSES = 27;
+	public static final int WEAVING_TYPE = 28;
+	public static final int MATCHING_SHADOW = 29;
+	public static final int IMPLEMENTING_ON_SHADOW = 30;
+	public static final int MATCHING_POINTCUT = 31;
+	public static final int MUNGING_WITH = 32;
+	public static final int PROCESSING_ATASPECTJTYPE_MUNGERS_ONLY = 33;
 
 	// phase names
 	public static final String[] PHASE_NAMES = new String[] { "batch building", "incrementally building",
 			"processing compilation unit", "resolving types defined in compilation unit",
 			"analysing types defined in compilation unit", "generating unwoven code for type defined in compilation unit",
-			"completing type bindings", "processing declare parents", "checking and setting imports", "connecting type hierarchy",
+			"completing type bindings", "processing declare parents", "checking and setting imports", "connecting type hierarchy 1",
 			"building fields and methods", "collecting itds and declares", "processing declare annotations",
 			"weaving intertype declarations", "resolving pointcut declarations", "adding declare warning and errors",
 			"validating @AspectJ annotations", "creating accessors for inlining", "adding @AspectJ annotations",
 			"fixing super calls in ITDs in interface context", "fixing super calls in ITDs",
-			"optimizing thisJoinPoint calls",
+			"optimizing thisJoinPoint calls", "connecting type hierarchy 2",
 
 			// BACK END
 

org.aspectj.ajdt.core/src/main/java/org/aspectj/ajdt/internal/compiler/lookup/AjLookupEnvironment.java

@@ -146,19 +146,12 @@ public void completeTypeBindings() {
 		stepCompleted = CHECK_AND_SET_IMPORTS;
 
 		for (int i = lastCompletedUnitIndex + 1; i <= lastUnitIndex; i++) {
-			ContextToken tok = CompilationAndWeavingContext.enteringPhase(CompilationAndWeavingContext.CONNECTING_TYPE_HIERARCHY,
+			ContextToken tok = CompilationAndWeavingContext.enteringPhase(CompilationAndWeavingContext.CONNECTING_TYPE_HIERARCHY1,
 					units[i].compilationResult.fileName);
 			units[i].scope.connectTypeHierarchy1();
 			CompilationAndWeavingContext.leavingPhase(tok);
 		}
 		stepCompleted = CONNECT_TYPE_HIERARCHY1;
-		for (int i = lastCompletedUnitIndex + 1; i <= lastUnitIndex; i++) {
-			ContextToken tok = CompilationAndWeavingContext.enteringPhase(CompilationAndWeavingContext.CONNECTING_TYPE_HIERARCHY,
-					units[i].compilationResult.fileName);
-			units[i].scope.connectTypeHierarchy2();
-			CompilationAndWeavingContext.leavingPhase(tok);
-		}
-		stepCompleted = CONNECT_TYPE_HIERARCHY2;
 		for (int i = lastCompletedUnitIndex + 1; i <= lastUnitIndex; i++) {
 			ContextToken tok = CompilationAndWeavingContext.enteringPhase(CompilationAndWeavingContext.BUILDING_FIELDS_AND_METHODS,
 					units[i].compilationResult.fileName);
@@ -300,11 +293,23 @@ public void completeTypeBindings() {
 			}
 		}
 
+		stepCompleted = BUILD_FIELDS_AND_METHODS;
+
+		// TODO:
+		//   Is it correct to put the call to CompilationUnitScope.connectTypeHierarchy2() here,
+		//   or should it occur earlier?
+		for (int i = lastCompletedUnitIndex + 1; i <= lastUnitIndex; i++) {
+			ContextToken tok = CompilationAndWeavingContext.enteringPhase(CompilationAndWeavingContext.CONNECTING_TYPE_HIERARCHY2,
+				units[i].compilationResult.fileName);
+			units[i].scope.connectTypeHierarchy2();
+			CompilationAndWeavingContext.leavingPhase(tok);
+		}
+		stepCompleted = CONNECT_TYPE_HIERARCHY2;
+
 		for (int i = lastCompletedUnitIndex + 1; i <= lastUnitIndex; i++) {
 			units[i] = null; // release unnecessary reference to the parsed unit
 		}
 
-		stepCompleted = BUILD_FIELDS_AND_METHODS;
 		lastCompletedUnitIndex = lastUnitIndex;
 		AsmManager.setCompletingTypeBindings(false);
 		factory.getWorld().getCrosscuttingMembersSet().verify();