From c37cc870d18123be42a8cd508722217a46c1da60 Mon Sep 17 00:00:00 2001 From: acolyer Date: Fri, 23 Jan 2004 11:49:16 +0000 Subject: [PATCH] spring cleaning in advance of 1.2 enhancements: removed unused imports, local variables, and private methods (still there, just commented out). --- .../src/org/aspectj/ajdt/ajc/AjdtCommand.java | 2 +- .../compiler/ast/AccessForInlineVisitor.java | 6 +-- .../compiler/ast/AdviceDeclaration.java | 2 +- .../internal/compiler/ast/AspectClinit.java | 4 +- .../compiler/ast/AspectDeclaration.java | 16 ++++---- .../compiler/ast/DeclareDeclaration.java | 2 +- .../compiler/ast/IfMethodDeclaration.java | 2 +- .../ast/InterTypeConstructorDeclaration.java | 10 ++--- .../ast/InterTypeMethodDeclaration.java | 4 +- .../compiler/ast/PointcutDeclaration.java | 4 +- .../internal/compiler/ast/ProceedVisitor.java | 2 +- .../compiler/ast/ThisJoinPointVisitor.java | 2 +- .../compiler/lookup/AjLookupEnvironment.java | 8 ++-- .../AsmInterTypeRelationshipProvider.java | 22 +++++----- .../compiler/lookup/EclipseFactory.java | 2 +- .../compiler/lookup/EclipseScope.java | 14 +++---- .../compiler/lookup/EclipseShadow.java | 14 +++---- .../compiler/lookup/EclipseSourceType.java | 6 +-- .../compiler/lookup/EclipseTypeMunger.java | 4 +- .../lookup/InlineAccessFieldBinding.java | 4 +- .../lookup/InterTypeMemberFinder.java | 14 +++---- .../lookup/PrivilegedFieldBinding.java | 4 +- .../compiler/lookup/PrivilegedHandler.java | 2 +- .../internal/compiler/parser/AjParser.java | 8 ++-- .../compiler/problem/AjProblemReporter.java | 2 +- .../internal/core/builder/AjBuildManager.java | 10 ++--- .../core/builder/AsmElementFormatter.java | 6 +-- .../core/builder/AsmHierarchyBuilder.java | 34 ++++++++-------- .../builder/EmacsStructureModelManager.java | 40 +++++++++---------- .../core/builder/StatefulNameEnvironment.java | 2 +- .../src/org/aspectj/tools/ajc/Main.java | 26 ++++++------ .../aspectj/ajdt/ajc/AjdtCommandTestCase.java | 18 ++++----- .../ajdt/ajc/BuildArgParserTestCase.java | 14 +++---- .../compiler/batch/BasicCommandTestCase.java | 8 ++-- .../compiler/batch/CommandTestCase.java | 2 +- .../compiler/batch/IncrementalCase.java | 8 ++-- .../core/builder/AjBuildManagerTest.java | 12 +++--- .../ant/taskdefs/Ajc11CompilerAdapter.java | 6 +-- .../org/aspectj/tools/ant/taskdefs/Ajc2.java | 4 +- .../aspectj/tools/ant/taskdefs/AjcTask.java | 12 +++--- 40 files changed, 181 insertions(+), 181 deletions(-) diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/ajc/AjdtCommand.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/ajc/AjdtCommand.java index 9527d635c..a04231cd2 100644 --- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/ajc/AjdtCommand.java +++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/ajc/AjdtCommand.java @@ -26,7 +26,7 @@ public class AjdtCommand implements ICommand { /** Message String for any AbortException thrown from ICommand API's */ public static final String ABORT_MESSAGE = "ABORT"; - private boolean canRepeatCommand = true; +// private boolean canRepeatCommand = true; AjBuildManager buildManager = null; String[] savedArgs = null; diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/AccessForInlineVisitor.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/AccessForInlineVisitor.java index 6580e6e21..d4af2905c 100644 --- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/AccessForInlineVisitor.java +++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/AccessForInlineVisitor.java @@ -13,7 +13,7 @@ package org.aspectj.ajdt.internal.compiler.ast; -import java.util.Arrays; +//import java.util.Arrays; import org.aspectj.ajdt.internal.compiler.lookup.EclipseFactory; import org.aspectj.ajdt.internal.compiler.lookup.InlineAccessFieldBinding; @@ -167,7 +167,7 @@ public class AccessForInlineVisitor extends ASTVisitor { } - ResolvedMember m = world.makeResolvedMember(binding); + ResolvedMember m = EclipseFactory.makeResolvedMember(binding); if (inAspect.accessForInline.containsKey(m)) return (MethodBinding)inAspect.accessForInline.get(m); MethodBinding ret = world.makeMethodBinding( AjcMemberMaker.inlineAccessMethodForMethod(inAspect.typeX, m) @@ -186,7 +186,7 @@ public class AccessForInlineVisitor extends ASTVisitor { } private MethodBinding getSuperAccessMethod(MethodBinding binding) { - ResolvedMember m = world.makeResolvedMember(binding); + ResolvedMember m = EclipseFactory.makeResolvedMember(binding); ResolvedMember superAccessMember = AjcMemberMaker.superAccessMethod(inAspect.typeX, m); if (inAspect.superAccessForInline.containsKey(superAccessMember)) { return ((SuperAccessMethodPair)inAspect.superAccessForInline.get(superAccessMember)).accessMethod; diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/AdviceDeclaration.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/AdviceDeclaration.java index 40955af6b..6b6b801f5 100644 --- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/AdviceDeclaration.java +++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/AdviceDeclaration.java @@ -131,7 +131,7 @@ public class AdviceDeclaration extends MethodDeclaration { if (kind == AdviceKind.Around) { int n = proceedCalls.size(); - EclipseFactory world = EclipseFactory.fromScopeLookupEnvironment(upperScope); +// EclipseFactory world = EclipseFactory.fromScopeLookupEnvironment(upperScope); //System.err.println("access to: " + Arrays.asList(handler.getMembers())); diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/AspectClinit.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/AspectClinit.java index ebead22eb..4c1ed2ff1 100644 --- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/AspectClinit.java +++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/AspectClinit.java @@ -55,7 +55,7 @@ public class AspectClinit extends Clinit { codeStream.invokestatic(world.makeMethodBindingForCall( AjcMemberMaker.ajcPreClinitMethod( - world.fromBinding(classScope.referenceContext.binding) + EclipseFactory.fromBinding(classScope.referenceContext.binding) ))); } super.generateSyntheticCode(classScope, codeStream); @@ -71,7 +71,7 @@ public class AspectClinit extends Clinit { codeStream.invokestatic(world.makeMethodBindingForCall( AjcMemberMaker.ajcPostClinitMethod( - world.fromBinding(classScope.referenceContext.binding) + EclipseFactory.fromBinding(classScope.referenceContext.binding) ))); } diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/AspectDeclaration.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/AspectDeclaration.java index a823f10b0..f2f343a69 100644 --- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/AspectDeclaration.java +++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/AspectDeclaration.java @@ -21,14 +21,14 @@ import org.aspectj.weaver.*; import org.aspectj.weaver.patterns.*; import org.eclipse.jdt.internal.compiler.ClassFile; import org.eclipse.jdt.internal.compiler.CompilationResult; -import org.eclipse.jdt.internal.compiler.ast.AbstractMethodDeclaration; +//import org.eclipse.jdt.internal.compiler.ast.AbstractMethodDeclaration; import org.eclipse.jdt.internal.compiler.ast.Clinit; -import org.eclipse.jdt.internal.compiler.ast.ConstructorDeclaration; +//import org.eclipse.jdt.internal.compiler.ast.ConstructorDeclaration; import org.eclipse.jdt.internal.compiler.ast.TypeDeclaration; import org.eclipse.jdt.internal.compiler.codegen.CodeStream; import org.eclipse.jdt.internal.compiler.codegen.Label; import org.eclipse.jdt.internal.compiler.lookup.*; -import org.eclipse.jdt.internal.compiler.parser.Parser; +//import org.eclipse.jdt.internal.compiler.parser.Parser; // (we used to...) making all aspects member types avoids a nasty hierarchy pain @@ -286,7 +286,7 @@ public class AspectDeclaration extends TypeDeclaration { } private void generateMethod(ClassFile classFile, MethodBinding methodBinding, BodyGenerator gen) { - EclipseFactory world = EclipseFactory.fromScopeLookupEnvironment(this.scope); +// EclipseFactory world = EclipseFactory.fromScopeLookupEnvironment(this.scope); classFile.generateMethodInfoHeader(methodBinding); int methodAttributeOffset = classFile.contentsOffset; int attributeNumber = classFile.generateMethodInfoAttribute(methodBinding, AstUtil.getAjSyntheticAttribute()); @@ -346,7 +346,7 @@ public class AspectDeclaration extends TypeDeclaration { { final EclipseFactory world = EclipseFactory.fromScopeLookupEnvironment(this.scope); generateMethod(classFile, world.makeMethodBinding(AjcMemberMaker.perCflowPush( - world.fromBinding(binding))), + EclipseFactory.fromBinding(binding))), new BodyGenerator() { public void generate(CodeStream codeStream) { // body starts here @@ -378,7 +378,7 @@ public class AspectDeclaration extends TypeDeclaration { { final EclipseFactory world = EclipseFactory.fromScopeLookupEnvironment(this.scope); generateMethod(classFile, world.makeMethodBinding(AjcMemberMaker.ajcPreClinitMethod( - world.fromBinding(binding))), + EclipseFactory.fromBinding(binding))), new BodyGenerator() { public void generate(CodeStream codeStream) { // body starts here @@ -480,7 +480,7 @@ public class AspectDeclaration extends TypeDeclaration { final TypeBinding interfaceType) { final EclipseFactory world = EclipseFactory.fromScopeLookupEnvironment(this.scope); - generateMethod(classFile, AjcMemberMaker.perObjectBind(world.fromBinding(binding)), + generateMethod(classFile, AjcMemberMaker.perObjectBind(EclipseFactory.fromBinding(binding)), new BodyGenerator() { public void generate(CodeStream codeStream) { // body starts here @@ -563,7 +563,7 @@ public class AspectDeclaration extends TypeDeclaration { { final EclipseFactory world = EclipseFactory.fromScopeLookupEnvironment(this.scope); generateMethod(classFile, world.makeMethodBinding(AjcMemberMaker.ajcPostClinitMethod( - world.fromBinding(binding))), + EclipseFactory.fromBinding(binding))), new BodyGenerator() { public void generate(CodeStream codeStream) { // body starts here diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/DeclareDeclaration.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/DeclareDeclaration.java index 34791014d..b1a57c116 100644 --- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/DeclareDeclaration.java +++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/DeclareDeclaration.java @@ -13,7 +13,7 @@ package org.aspectj.ajdt.internal.compiler.ast; -import java.util.List; +//import java.util.List; import org.aspectj.ajdt.internal.compiler.lookup.EclipseScope; import org.aspectj.weaver.AjAttribute; diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/IfMethodDeclaration.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/IfMethodDeclaration.java index e770dfc23..5103a8a78 100644 --- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/IfMethodDeclaration.java +++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/IfMethodDeclaration.java @@ -22,7 +22,7 @@ import org.eclipse.jdt.internal.compiler.ClassFile; import org.eclipse.jdt.internal.compiler.CompilationResult; import org.eclipse.jdt.internal.compiler.ast.CompilationUnitDeclaration; import org.eclipse.jdt.internal.compiler.ast.MethodDeclaration; -import org.eclipse.jdt.internal.compiler.lookup.ClassScope; +//import org.eclipse.jdt.internal.compiler.lookup.ClassScope; import org.eclipse.jdt.internal.compiler.parser.Parser; public class IfMethodDeclaration extends MethodDeclaration { diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/InterTypeConstructorDeclaration.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/InterTypeConstructorDeclaration.java index 59d139121..0f189c34c 100644 --- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/InterTypeConstructorDeclaration.java +++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/InterTypeConstructorDeclaration.java @@ -92,7 +92,7 @@ public class InterTypeConstructorDeclaration extends InterTypeDeclaration { pre.binding = world.makeMethodBinding( AjcMemberMaker.preIntroducedConstructor(aspectTypeX, targetTypeX, - world.fromBindings(binding.parameters))); + EclipseFactory.fromBindings(binding.parameters))); pre.bindArguments(); pre.bindThrownExceptions(); @@ -140,7 +140,7 @@ public class InterTypeConstructorDeclaration extends InterTypeDeclaration { Expression arg = explicitConstructorCall.arguments[i]; ResolvedMember conversionMethod = - AjcMemberMaker.toObjectConversionMethod(world.fromBinding(explicitConstructorCall.binding.parameters[i])); + AjcMemberMaker.toObjectConversionMethod(EclipseFactory.fromBinding(explicitConstructorCall.binding.parameters[i])); if (conversionMethod != null) { arg = new KnownMessageSend(world.makeMethodBindingForCall(conversionMethod), new CastExpression(new NullLiteral(0, 0), @@ -154,7 +154,7 @@ public class InterTypeConstructorDeclaration extends InterTypeDeclaration { LocalVariableBinding binding = pre.arguments[i].binding; Expression arg = AstUtil.makeResolvedLocalVariableReference(binding); ResolvedMember conversionMethod = - AjcMemberMaker.toObjectConversionMethod(world.fromBinding(binding.type)); + AjcMemberMaker.toObjectConversionMethod(EclipseFactory.fromBinding(binding.type)); if (conversionMethod != null) { arg = new KnownMessageSend(world.makeMethodBindingForCall(conversionMethod), new CastExpression(new NullLiteral(0, 0), @@ -208,7 +208,7 @@ public class InterTypeConstructorDeclaration extends InterTypeDeclaration { ResolvedTypeX declaringTypeX = world.fromEclipse(onTypeBinding); ResolvedTypeX aspectType = world.fromEclipse(classScope.referenceContext.binding); - ResolvedMember bindingAsMember = world.makeResolvedMember(binding); + ResolvedMember bindingAsMember = EclipseFactory.makeResolvedMember(binding); ResolvedMember signature = new ResolvedMember(Member.CONSTRUCTOR, declaringTypeX, declaredModifiers, @@ -240,7 +240,7 @@ public class InterTypeConstructorDeclaration extends InterTypeDeclaration { ((NewConstructorTypeMunger)munger).setExplicitConstructor( - world.makeResolvedMember(explicitConstructor)); + EclipseFactory.makeResolvedMember(explicitConstructor)); } else { ((NewConstructorTypeMunger)munger).setExplicitConstructor( new ResolvedMember(Member.CONSTRUCTOR, diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/InterTypeMethodDeclaration.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/InterTypeMethodDeclaration.java index 3babcfbf0..d5bf73da6 100644 --- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/InterTypeMethodDeclaration.java +++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/InterTypeMethodDeclaration.java @@ -23,7 +23,7 @@ import org.eclipse.jdt.internal.compiler.CompilationResult; import org.eclipse.jdt.internal.compiler.ast.CompilationUnitDeclaration; import org.eclipse.jdt.internal.compiler.ast.TypeReference; import org.eclipse.jdt.internal.compiler.codegen.CodeStream; -import org.eclipse.jdt.internal.compiler.flow.FlowContext; +//import org.eclipse.jdt.internal.compiler.flow.FlowContext; import org.eclipse.jdt.internal.compiler.flow.FlowInfo; import org.eclipse.jdt.internal.compiler.flow.InitializationFlowContext; import org.eclipse.jdt.internal.compiler.lookup.*; @@ -177,7 +177,7 @@ public class InterTypeMethodDeclaration extends InterTypeDeclaration { else resolvedPosition++; } - TypeBinding type; +// TypeBinding type; if (methodBinding.isStatic()) codeStream.invokestatic(methodBinding); else { diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/PointcutDeclaration.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/PointcutDeclaration.java index b55d3188f..b0595257e 100644 --- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/PointcutDeclaration.java +++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/PointcutDeclaration.java @@ -19,8 +19,8 @@ import org.aspectj.ajdt.internal.compiler.lookup.EclipseFactory; import org.aspectj.ajdt.internal.core.builder.EclipseSourceContext; import org.aspectj.weaver.AjAttribute; import org.aspectj.weaver.ResolvedPointcutDefinition; -import org.aspectj.weaver.ResolvedTypeX; -import org.aspectj.weaver.TypeX; +//import org.aspectj.weaver.ResolvedTypeX; +//import org.aspectj.weaver.TypeX; import org.aspectj.weaver.patterns.Pointcut; import org.eclipse.jdt.internal.compiler.ClassFile; import org.eclipse.jdt.internal.compiler.CompilationResult; diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/ProceedVisitor.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/ProceedVisitor.java index 4c0d2bc9d..a63ee3818 100644 --- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/ProceedVisitor.java +++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/ProceedVisitor.java @@ -204,7 +204,7 @@ public class ProceedVisitor extends ASTVisitor { } private void removeParameter(int indexToRemove) { - TypeBinding[] parameters = method.binding.parameters; +// TypeBinding[] parameters = method.binding.parameters; method.scope.locals = removeLocalBinding(indexToRemove, method.scope.locals); method.binding.parameters = removeParameter(indexToRemove, method.binding.parameters); } diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/ThisJoinPointVisitor.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/ThisJoinPointVisitor.java index 7cfbc8ea3..261e14af9 100644 --- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/ThisJoinPointVisitor.java +++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/ThisJoinPointVisitor.java @@ -208,7 +208,7 @@ public class ThisJoinPointVisitor extends ASTVisitor { } private void removeParameter(int indexToRemove) { - TypeBinding[] parameters = method.binding.parameters; +// TypeBinding[] parameters = method.binding.parameters; method.scope.locals = removeLocalBinding(indexToRemove, method.scope.locals); method.scope.localIndex -= 1; method.binding.parameters = removeParameter(indexToRemove, method.binding.parameters); diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/AjLookupEnvironment.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/AjLookupEnvironment.java index b3af379ee..4d0e4cae4 100644 --- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/AjLookupEnvironment.java +++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/AjLookupEnvironment.java @@ -16,11 +16,11 @@ package org.aspectj.ajdt.internal.compiler.lookup; import java.util.*; import org.aspectj.ajdt.internal.compiler.ast.AspectDeclaration; -import org.aspectj.ajdt.internal.compiler.ast.DeclareDeclaration; +//import org.aspectj.ajdt.internal.compiler.ast.DeclareDeclaration; import org.aspectj.ajdt.internal.compiler.ast.PointcutDeclaration; -import org.aspectj.asm.*; -import org.aspectj.asm.IProgramElement; -import org.aspectj.asm.internal.Relationship; +//import org.aspectj.asm.*; +//import org.aspectj.asm.IProgramElement; +//import org.aspectj.asm.internal.Relationship; import org.aspectj.bridge.IMessage; import org.aspectj.weaver.*; import org.aspectj.weaver.patterns.*; diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/AsmInterTypeRelationshipProvider.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/AsmInterTypeRelationshipProvider.java index 88a9e1953..9bb093190 100644 --- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/AsmInterTypeRelationshipProvider.java +++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/AsmInterTypeRelationshipProvider.java @@ -12,7 +12,7 @@ package org.aspectj.ajdt.internal.compiler.lookup; -import java.io.IOException; +//import java.io.IOException; import org.aspectj.asm.*; import org.aspectj.asm.internal.ProgramElement; @@ -30,16 +30,16 @@ public class AsmInterTypeRelationshipProvider { ResolvedTypeX onType, EclipseTypeMunger munger) { - IProgramElement.Kind kind = IProgramElement.Kind.ERROR; - if (munger.getMunger().getKind() == ResolvedTypeMunger.Field) { - kind = IProgramElement.Kind.INTER_TYPE_FIELD; - } else if (munger.getMunger().getKind() == ResolvedTypeMunger.Constructor) { - kind = IProgramElement.Kind.INTER_TYPE_CONSTRUCTOR; - } else if (munger.getMunger().getKind() == ResolvedTypeMunger.Method) { - kind = IProgramElement.Kind.INTER_TYPE_METHOD; - } else if (munger.getMunger().getKind() == ResolvedTypeMunger.Parent) { - kind = IProgramElement.Kind.INTER_TYPE_PARENT; - } +// IProgramElement.Kind kind = IProgramElement.Kind.ERROR; +// if (munger.getMunger().getKind() == ResolvedTypeMunger.Field) { +// kind = IProgramElement.Kind.INTER_TYPE_FIELD; +// } else if (munger.getMunger().getKind() == ResolvedTypeMunger.Constructor) { +// kind = IProgramElement.Kind.INTER_TYPE_CONSTRUCTOR; +// } else if (munger.getMunger().getKind() == ResolvedTypeMunger.Method) { +// kind = IProgramElement.Kind.INTER_TYPE_METHOD; +// } else if (munger.getMunger().getKind() == ResolvedTypeMunger.Parent) { +// kind = IProgramElement.Kind.INTER_TYPE_PARENT; +// } if (munger.getSourceLocation() != null && munger.getSourceLocation() != null) { diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/EclipseFactory.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/EclipseFactory.java index ba6d39ce9..1a4e71fe0 100644 --- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/EclipseFactory.java +++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/EclipseFactory.java @@ -37,7 +37,7 @@ public class EclipseFactory { private Map/*TypeX, TypeBinding*/ typexToBinding = new HashMap(); //XXX currently unused - private Map/*TypeBinding, ResolvedTypeX*/ bindingToResolvedTypeX = new HashMap(); +// private Map/*TypeBinding, ResolvedTypeX*/ bindingToResolvedTypeX = new HashMap(); public static EclipseFactory fromLookupEnvironment(LookupEnvironment env) { AjLookupEnvironment aenv = (AjLookupEnvironment)env; diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/EclipseScope.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/EclipseScope.java index 58a14aa32..1bccc0e24 100644 --- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/EclipseScope.java +++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/EclipseScope.java @@ -69,7 +69,7 @@ public class EclipseScope implements IScope { //System.err.println("binding: " + b); // Binding(tokens, bits & RestrictiveFlagMASK, this) - return world.fromBinding(b); + return EclipseFactory.fromBinding(b); /* computeImports(); @@ -114,12 +114,12 @@ public class EclipseScope implements IScope { } - private ResolvedTypeX resolveVisible(String name) { - ResolvedTypeX found = world.getWorld().resolve(TypeX.forName(name), true); - if (found == ResolvedTypeX.MISSING) return found; - if (ResolvedTypeX.isVisible(found.getModifiers(), found, enclosingType)) return found; - return ResolvedTypeX.MISSING; - } +// private ResolvedTypeX resolveVisible(String name) { +// ResolvedTypeX found = world.getWorld().resolve(TypeX.forName(name), true); +// if (found == ResolvedTypeX.MISSING) return found; +// if (ResolvedTypeX.isVisible(found.getModifiers(), found, enclosingType)) return found; +// return ResolvedTypeX.MISSING; +// } // public TypeX lookupType(String name, IHasPosition location) { diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/EclipseShadow.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/EclipseShadow.java index 99abeedf9..5aef59393 100644 --- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/EclipseShadow.java +++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/EclipseShadow.java @@ -50,9 +50,9 @@ public class EclipseShadow extends Shadow { public TypeX getEnclosingType() { if (context instanceof TypeDeclaration) { - return world.fromBinding(((TypeDeclaration)context).binding); + return EclipseFactory.fromBinding(((TypeDeclaration)context).binding); } else if (context instanceof AbstractMethodDeclaration) { - return world.fromBinding(((AbstractMethodDeclaration)context).binding.declaringClass); + return EclipseFactory.fromBinding(((AbstractMethodDeclaration)context).binding.declaringClass); } else { return ResolvedTypeX.MISSING; } @@ -68,7 +68,7 @@ public class EclipseShadow extends Shadow { return new Member(Member.STATIC_INITIALIZATION, getEnclosingType(), 0, ResolvedTypeX.VOID, "", TypeX.NONE); } else if (context instanceof AbstractMethodDeclaration) { - return world.makeResolvedMember(((AbstractMethodDeclaration)context).binding); + return EclipseFactory.makeResolvedMember(((AbstractMethodDeclaration)context).binding); } else { return null; } @@ -110,12 +110,12 @@ public class EclipseShadow extends Shadow { if (astNode instanceof AllocationExpression) { AllocationExpression e = (AllocationExpression)astNode; return new EclipseShadow(world, Shadow.ConstructorCall, - world.makeResolvedMember(e.binding), astNode, context); + EclipseFactory.makeResolvedMember(e.binding), astNode, context); } else if (astNode instanceof MessageSend) { MessageSend e = (MessageSend)astNode; if (e.isSuperAccess()) return null; // super calls don't have shadows return new EclipseShadow(world, Shadow.MethodCall, - world.makeResolvedMember(e.binding), astNode, context); + EclipseFactory.makeResolvedMember(e.binding), astNode, context); } else if (astNode instanceof ExplicitConstructorCall) { //??? these should be ignored, they don't have shadows return null; @@ -143,11 +143,11 @@ public class EclipseShadow extends Shadow { //throw new RuntimeException("unimplemented: " + e); } return new EclipseShadow(world, kind, - world.makeResolvedMember(e.binding), astNode, context); + EclipseFactory.makeResolvedMember(e.binding), astNode, context); } else if (astNode instanceof TypeDeclaration) { return new EclipseShadow(world, Shadow.StaticInitialization, new Member(Member.STATIC_INITIALIZATION, - world.fromBinding(((TypeDeclaration)astNode).binding), 0, + EclipseFactory.fromBinding(((TypeDeclaration)astNode).binding), 0, ResolvedTypeX.VOID, "", TypeX.NONE), astNode, context); } else { diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/EclipseSourceType.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/EclipseSourceType.java index 60f317f1f..2667909bc 100644 --- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/EclipseSourceType.java +++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/EclipseSourceType.java @@ -18,7 +18,7 @@ import java.util.*; import org.aspectj.ajdt.internal.compiler.ast.*; import org.aspectj.ajdt.internal.core.builder.EclipseSourceContext; import org.aspectj.bridge.IMessage; -import org.aspectj.bridge.ISourceLocation; +//import org.aspectj.bridge.ISourceLocation; import org.aspectj.weaver.*; import org.aspectj.weaver.patterns.PerClause; import org.aspectj.weaver.patterns.PerSingleton; @@ -107,7 +107,7 @@ public class EclipseSourceType extends ResolvedTypeX.ConcreteName { continue; } else { if (amd.binding == null || !amd.binding.isValidBinding()) continue; - declaredMethods.add(eclipseWorld().makeResolvedMember(amd.binding)); + declaredMethods.add(EclipseFactory.makeResolvedMember(amd.binding)); } } } @@ -115,7 +115,7 @@ public class EclipseSourceType extends ResolvedTypeX.ConcreteName { FieldBinding[] fields = binding.fields(); for (int i=0, len=fields.length; i < len; i++) { FieldBinding f = fields[i]; - declaredFields.add(eclipseWorld().makeResolvedMember(f)); + declaredFields.add(EclipseFactory.makeResolvedMember(f)); } this.declaredPointcuts = (ResolvedPointcutDefinition[]) diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/EclipseTypeMunger.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/EclipseTypeMunger.java index 2c4b9489c..33a4f2a78 100644 --- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/EclipseTypeMunger.java +++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/EclipseTypeMunger.java @@ -22,11 +22,11 @@ import org.aspectj.weaver.NewFieldTypeMunger; import org.aspectj.weaver.NewMethodTypeMunger; import org.aspectj.weaver.ResolvedTypeMunger; import org.aspectj.weaver.ResolvedTypeX; -import org.aspectj.weaver.TypeX; +//import org.aspectj.weaver.TypeX; import org.eclipse.jdt.internal.compiler.ast.AbstractMethodDeclaration; import org.eclipse.jdt.internal.compiler.lookup.FieldBinding; import org.eclipse.jdt.internal.compiler.lookup.MethodBinding; -import org.eclipse.jdt.internal.compiler.lookup.ReferenceBinding; +//import org.eclipse.jdt.internal.compiler.lookup.ReferenceBinding; import org.eclipse.jdt.internal.compiler.lookup.SourceTypeBinding; diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/InlineAccessFieldBinding.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/InlineAccessFieldBinding.java index d1f3b730f..f0b844903 100644 --- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/InlineAccessFieldBinding.java +++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/InlineAccessFieldBinding.java @@ -46,11 +46,11 @@ public class InlineAccessFieldBinding extends FieldBinding { this.reader = new SimpleSyntheticAccessMethodBinding( inAspect.factory.makeMethodBinding( AjcMemberMaker.inlineAccessMethodForFieldGet( - inAspect.typeX, inAspect.factory.makeResolvedMember(baseField) + inAspect.typeX, EclipseFactory.makeResolvedMember(baseField) ))); this.writer = new SimpleSyntheticAccessMethodBinding(inAspect.factory.makeMethodBinding( AjcMemberMaker.inlineAccessMethodForFieldSet( - inAspect.typeX, inAspect.factory.makeResolvedMember(baseField) + inAspect.typeX, EclipseFactory.makeResolvedMember(baseField) ))); this.constant = ASTNode.NotAConstant; diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/InterTypeMemberFinder.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/InterTypeMemberFinder.java index ae7da9032..eaa81c8da 100644 --- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/InterTypeMemberFinder.java +++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/InterTypeMemberFinder.java @@ -233,13 +233,13 @@ public class InterTypeMemberFinder implements IMemberFinder { // } // } - private ReferenceBinding getTargetType(MethodBinding m2) { - if (m2 instanceof InterTypeMethodBinding) { - return ((InterTypeMethodBinding)m2).getTargetType(); - } - - return m2.declaringClass; - } +// private ReferenceBinding getTargetType(MethodBinding m2) { +// if (m2 instanceof InterTypeMethodBinding) { +// return ((InterTypeMethodBinding)m2).getTargetType(); +// } +// +// return m2.declaringClass; +// } diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/PrivilegedFieldBinding.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/PrivilegedFieldBinding.java index adedd1dbb..af0db72c2 100644 --- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/PrivilegedFieldBinding.java +++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/PrivilegedFieldBinding.java @@ -35,11 +35,11 @@ public class PrivilegedFieldBinding extends FieldBinding { this.reader = new SimpleSyntheticAccessMethodBinding( inAspect.factory.makeMethodBinding( AjcMemberMaker.privilegedAccessMethodForFieldGet( - inAspect.typeX, inAspect.factory.makeResolvedMember(baseField) + inAspect.typeX, EclipseFactory.makeResolvedMember(baseField) ))); this.writer = new SimpleSyntheticAccessMethodBinding(inAspect.factory.makeMethodBinding( AjcMemberMaker.privilegedAccessMethodForFieldSet( - inAspect.typeX, inAspect.factory.makeResolvedMember(baseField) + inAspect.typeX, EclipseFactory.makeResolvedMember(baseField) ))); this.constant = ASTNode.NotAConstant; diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/PrivilegedHandler.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/PrivilegedHandler.java index 6ee798eb8..f76569475 100644 --- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/PrivilegedHandler.java +++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/lookup/PrivilegedHandler.java @@ -45,7 +45,7 @@ public class PrivilegedHandler implements IPrivilegedHandler { } public FieldBinding getPrivilegedAccessField(FieldBinding baseField, ASTNode location) { - ResolvedMember key = inAspect.factory.makeResolvedMember(baseField); + ResolvedMember key = EclipseFactory.makeResolvedMember(baseField); if (accessors.containsKey(key)) return (FieldBinding)accessors.get(key); FieldBinding ret = new PrivilegedFieldBinding(inAspect, baseField); checkWeaveAccess(key.getDeclaringType(), location); diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/parser/AjParser.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/parser/AjParser.java index be36f8dca..87232ada8 100644 --- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/parser/AjParser.java +++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/parser/AjParser.java @@ -40,7 +40,7 @@ import org.eclipse.jdt.internal.compiler.ast.ExplicitConstructorCall; import org.eclipse.jdt.internal.compiler.ast.Expression; import org.eclipse.jdt.internal.compiler.ast.Literal; import org.eclipse.jdt.internal.compiler.ast.MessageSend; -import org.eclipse.jdt.internal.compiler.ast.OperatorExpression; +//import org.eclipse.jdt.internal.compiler.ast.OperatorExpression; import org.eclipse.jdt.internal.compiler.ast.OperatorIds; import org.eclipse.jdt.internal.compiler.ast.TypeReference; import org.eclipse.jdt.internal.compiler.impl.CompilerOptions; @@ -223,8 +223,8 @@ public class AjParser extends Parser { // handle modifiers, only without privileged for now if (isPrivileged) { pos = eatIdentifier(); // eat the privileged - int end = (int) pos; - int start = (int) (pos >>> 32); +// int end = (int) pos; +// int start = (int) (pos >>> 32); aspectDecl.isPrivileged = true; //problemReporter().signalError(start, end, "privileged is unimplemented in 1.1alpha1"); } @@ -292,7 +292,7 @@ public class AjParser extends Parser { //the name long pos = identifierPositionStack[identifierPtr]; - int sourceEnd = (int) pos; +// int sourceEnd = (int) pos; ret.sourceStart = (int) (pos >>> 32); ret.selector = identifierStack[identifierPtr--]; identifierLengthPtr--; diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/problem/AjProblemReporter.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/problem/AjProblemReporter.java index 45ee2e081..2a4882e94 100644 --- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/problem/AjProblemReporter.java +++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/problem/AjProblemReporter.java @@ -175,7 +175,7 @@ public class AjProblemReporter extends ProblemReporter { CompilationResult unitResult) { if (severity != Ignore && DUMP_STACK) { - Thread.currentThread().dumpStack(); + Thread.dumpStack(); } super.handle( problemId, diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/core/builder/AjBuildManager.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/core/builder/AjBuildManager.java index 3727498c1..1ab91d09e 100644 --- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/core/builder/AjBuildManager.java +++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/core/builder/AjBuildManager.java @@ -22,7 +22,7 @@ import org.aspectj.ajdt.internal.compiler.lookup.*; import org.aspectj.ajdt.internal.compiler.parser.AjParser; import org.aspectj.ajdt.internal.compiler.problem.AjProblemReporter; import org.aspectj.asm.*; -import org.aspectj.asm.internal.*; +//import org.aspectj.asm.internal.*; import org.aspectj.asm.internal.ProgramElement; import org.aspectj.bridge.*; import org.aspectj.weaver.World; @@ -34,7 +34,7 @@ import org.eclipse.jdt.internal.compiler.batch.FileSystem; import org.eclipse.jdt.internal.compiler.env.*; import org.eclipse.jdt.internal.compiler.impl.CompilerOptions; import org.eclipse.jdt.internal.compiler.problem.DefaultProblemFactory; -import org.eclipse.jdt.internal.compiler.util.HashtableOfObject; +//import org.eclipse.jdt.internal.compiler.util.HashtableOfObject; public class AjBuildManager { static final boolean FAIL_IF_RUNTIME_NOT_FOUND = false; @@ -153,7 +153,7 @@ public class AjBuildManager { // have to tell state we succeeded or next is not incremental state.successfulCompile(buildConfig); - boolean weaved = weaveAndGenerateClassFiles(); + /*boolean weaved = */weaveAndGenerateClassFiles(); // if not weaved, then no-op build, no model changes // but always returns true // XXX weaved not in Mik's incremental @@ -194,7 +194,7 @@ public class AjBuildManager { bcelWeaver.addLibraryJarFile(f); } - String lintMode = buildConfig.getLintMode(); +// String lintMode = buildConfig.getLintMode(); if (buildConfig.getLintMode().equals(AjBuildConfig.AJLINT_DEFAULT)) { bcelWorld.getLint().loadDefaultProperties(); @@ -290,7 +290,7 @@ public class AjBuildManager { public CompilationUnit[] getCompilationUnits(String[] filenames, String[] encodings) { int fileCount = filenames.length; CompilationUnit[] units = new CompilationUnit[fileCount]; - HashtableOfObject knownFileNames = new HashtableOfObject(fileCount); +// HashtableOfObject knownFileNames = new HashtableOfObject(fileCount); String defaultEncoding = (String) buildConfig.getJavaOptions().get(CompilerOptions.OPTION_Encoding); if ("".equals(defaultEncoding)) //$NON-NLS-1$ diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/core/builder/AsmElementFormatter.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/core/builder/AsmElementFormatter.java index d6e02a1df..eddc09cde 100644 --- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/core/builder/AsmElementFormatter.java +++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/core/builder/AsmElementFormatter.java @@ -71,7 +71,7 @@ public class AsmElementFormatter { setParameters(methodDeclaration, node); } else if (methodDeclaration instanceof PointcutDeclaration) { - PointcutDeclaration pd = (PointcutDeclaration)methodDeclaration; +// PointcutDeclaration pd = (PointcutDeclaration)methodDeclaration; node.setKind(IProgramElement.Kind.POINTCUT); node.setName(translatePointcutName(new String(methodDeclaration.selector))); setParameters(methodDeclaration, node); @@ -123,10 +123,10 @@ public class AsmElementFormatter { node.setKind(IProgramElement.Kind.INTER_TYPE_FIELD); } else if (methodDeclaration instanceof InterTypeMethodDeclaration) { node.setKind(IProgramElement.Kind.INTER_TYPE_METHOD); - InterTypeMethodDeclaration itmd = (InterTypeMethodDeclaration)methodDeclaration; +// InterTypeMethodDeclaration itmd = (InterTypeMethodDeclaration)methodDeclaration; } else if (methodDeclaration instanceof InterTypeConstructorDeclaration) { node.setKind(IProgramElement.Kind.INTER_TYPE_CONSTRUCTOR); - InterTypeConstructorDeclaration itcd = (InterTypeConstructorDeclaration)methodDeclaration; +// InterTypeConstructorDeclaration itcd = (InterTypeConstructorDeclaration)methodDeclaration; } else { node.setKind(IProgramElement.Kind.ERROR); } diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/core/builder/AsmHierarchyBuilder.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/core/builder/AsmHierarchyBuilder.java index a594a9a56..3091094d7 100644 --- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/core/builder/AsmHierarchyBuilder.java +++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/core/builder/AsmHierarchyBuilder.java @@ -19,7 +19,7 @@ import java.util.*; import org.aspectj.ajdt.internal.compiler.ast.AspectDeclaration; import org.aspectj.ajdt.internal.compiler.lookup.EclipseFactory; import org.aspectj.asm.*; -import org.aspectj.asm.internal.*; +//import org.aspectj.asm.internal.*; import org.aspectj.asm.internal.ProgramElement; import org.aspectj.bridge.*; import org.aspectj.util.LangUtil; @@ -218,7 +218,7 @@ public class AsmHierarchyBuilder extends ASTVisitor { } public boolean visit(TypeDeclaration memberTypeDeclaration, BlockScope scope) { - String name = new String(memberTypeDeclaration.name); +// String name = new String(memberTypeDeclaration.name); String fullName = ""; if (memberTypeDeclaration.binding != null @@ -481,11 +481,11 @@ public class AsmHierarchyBuilder extends ASTVisitor { // AMC - overloaded set of methods to get start and end lines for // various ASTNode types. They have no common ancestor in the // hierarchy!! - private int getStartLine( AbstractVariableDeclaration avd ) { - return ProblemHandler.searchLineNumber( - currCompilationResult.lineSeparatorPositions, - avd.declarationSourceStart); - } +// private int getStartLine( AbstractVariableDeclaration avd ) { +// return ProblemHandler.searchLineNumber( +// currCompilationResult.lineSeparatorPositions, +// avd.declarationSourceStart); +// } // AMC - overloaded set of methods to get start and end lines for // various ASTNode types. They have no common ancestor in the @@ -499,11 +499,11 @@ public class AsmHierarchyBuilder extends ASTVisitor { // AMC - overloaded set of methods to get start and end lines for // various ASTNode types. They have no common ancestor in the // hierarchy!! - private int getStartLine( AbstractMethodDeclaration amd ){ - return ProblemHandler.searchLineNumber( - currCompilationResult.lineSeparatorPositions, - amd.declarationSourceStart); - } +// private int getStartLine( AbstractMethodDeclaration amd ){ +// return ProblemHandler.searchLineNumber( +// currCompilationResult.lineSeparatorPositions, +// amd.declarationSourceStart); +// } // AMC - overloaded set of methods to get start and end lines for // various ASTNode types. They have no common ancestor in the @@ -517,11 +517,11 @@ public class AsmHierarchyBuilder extends ASTVisitor { // AMC - overloaded set of methods to get start and end lines for // various ASTNode types. They have no common ancestor in the // hierarchy!! - private int getStartLine( TypeDeclaration td ){ - return ProblemHandler.searchLineNumber( - currCompilationResult.lineSeparatorPositions, - td.declarationSourceStart); - } +// private int getStartLine( TypeDeclaration td ){ +// return ProblemHandler.searchLineNumber( +// currCompilationResult.lineSeparatorPositions, +// td.declarationSourceStart); +// } // AMC - overloaded set of methods to get start and end lines for // various ASTNode types. They have no common ancestor in the diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/core/builder/EmacsStructureModelManager.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/core/builder/EmacsStructureModelManager.java index 4188adeb4..6dc8f40ef 100644 --- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/core/builder/EmacsStructureModelManager.java +++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/core/builder/EmacsStructureModelManager.java @@ -106,21 +106,21 @@ public class EmacsStructureModelManager { print(") "); } - private void printDecls(IRelationship node) { -// for (Iterator it = node.getTargets().iterator(); it.hasNext(); ) { -// // this ignores relations on the compile unit -// Object nodeObject = it.next(); -// throw new RuntimeException("unimplemented"); -//// if (nodeObject instanceof LinkNode) { -//// LinkNode child = (LinkNode)nodeObject; -//// if (//!child.getProgramElementNode().getKind().equals("stmnt") && -//// !child.getProgramElementNode().getKind().equals("")) { -//// printDecl(child.getProgramElementNode(), false); +// private void printDecls(IRelationship node) { +//// for (Iterator it = node.getTargets().iterator(); it.hasNext(); ) { +//// // this ignores relations on the compile unit +//// Object nodeObject = it.next(); +//// throw new RuntimeException("unimplemented"); +////// if (nodeObject instanceof LinkNode) { +////// LinkNode child = (LinkNode)nodeObject; +////// if (//!child.getProgramElementNode().getKind().equals("stmnt") && +////// !child.getProgramElementNode().getKind().equals("")) { ////// printDecl(child.getProgramElementNode(), false); -//// } -//// } -// } - } +//////// printDecl(child.getProgramElementNode(), false); +////// } +////// } +//// } +// } /** * @param structureNode can be a ProgramElementNode or a LinkNode @@ -216,12 +216,12 @@ public class EmacsStructureModelManager { subst(n, o, (in.substring(pos + o.length()))); } - private void lose(Error e) { - try { - print("(ERROR \"" + e.toString() + "\")"); - } - catch(Error ex) { } - } +// private void lose(Error e) { +// try { +// print("(ERROR \"" + e.toString() + "\")"); +// } +// catch(Error ex) { } +// } } } diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/core/builder/StatefulNameEnvironment.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/core/builder/StatefulNameEnvironment.java index af72b9a9a..bb54b5c3d 100644 --- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/core/builder/StatefulNameEnvironment.java +++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/core/builder/StatefulNameEnvironment.java @@ -13,7 +13,7 @@ package org.aspectj.ajdt.internal.core.builder; -import java.util.HashMap; +//import java.util.HashMap; import java.util.HashSet; import java.util.Iterator; import java.util.Map; diff --git a/org.aspectj.ajdt.core/src/org/aspectj/tools/ajc/Main.java b/org.aspectj.ajdt.core/src/org/aspectj/tools/ajc/Main.java index 2be51cb6d..e77a9f5a6 100644 --- a/org.aspectj.ajdt.core/src/org/aspectj/tools/ajc/Main.java +++ b/org.aspectj.ajdt.core/src/org/aspectj/tools/ajc/Main.java @@ -17,10 +17,10 @@ package org.aspectj.tools.ajc; import java.io.File; import java.io.IOException; import java.io.PrintStream; -import java.util.Arrays; -import java.util.List; - -import org.aspectj.ajdt.ajc.BuildArgParser; +//import java.util.Arrays; +//import java.util.List; +// +//import org.aspectj.ajdt.ajc.BuildArgParser; import org.aspectj.bridge.AbortException; import org.aspectj.bridge.ICommand; import org.aspectj.bridge.IMessage; @@ -225,12 +225,12 @@ public class Main { return; } try { - boolean verbose = (-1 != ("" + Arrays.asList(args)).indexOf("-verbose")); +// boolean verbose = (-1 != ("" + Arrays.asList(args)).indexOf("-verbose")); outer: while (true) { boolean passed = command.runCommand(args, holder); if (report(passed, holder) && controller.incremental()) { - final boolean onCommandLine = controller.commandLineIncremental(); +// final boolean onCommandLine = controller.commandLineIncremental(); while (controller.doRepeatCommand()) { holder.clearMessages(); if (controller.buildFresh()) { @@ -406,7 +406,7 @@ public class Main { * @return String rendering IMessage (never null) */ protected String render(IMessage message) { - IMessage.Kind kind = message.getKind(); +// IMessage.Kind kind = message.getKind(); StringBuffer sb = new StringBuffer(); String text = message.getMessage(); @@ -521,10 +521,10 @@ public class Main { */ public String[] init(String[] args, IMessageHandler sink) { running = true; - String[] unused; +// String[] unused; if (!LangUtil.isEmpty(args)) { String[][] options = LangUtil.copyStrings(OPTIONS); - unused = LangUtil.extractOptions(args, options); + /*unused = */LangUtil.extractOptions(args, options); incremental = (null != options[0][0]); if (null != options[1][0]) { File file = new File(options[1][1]); @@ -588,7 +588,7 @@ public class Main { } System.out.println(" press enter to recompile, r to rebuild, q to quit: "); System.out.flush(); - boolean doMore = false; +// boolean doMore = false; // seek for one q or a series of [\n\r]... do { int input = System.in.read(); @@ -628,10 +628,10 @@ public class Main { /** delay between filesystem checks, returning if quit is set */ protected void fileCheckDelay() { - final Thread thread = Thread.currentThread(); +// final Thread thread = Thread.currentThread(); long targetTime = System.currentTimeMillis() + delay; - long curTime; - while (targetTime > (curTime = System.currentTimeMillis())) { +// long curTime; + while (targetTime > System.currentTimeMillis()) { if (quit) { return; } diff --git a/org.aspectj.ajdt.core/testsrc/org/aspectj/ajdt/ajc/AjdtCommandTestCase.java b/org.aspectj.ajdt.core/testsrc/org/aspectj/ajdt/ajc/AjdtCommandTestCase.java index 19bdb1cf1..466078965 100644 --- a/org.aspectj.ajdt.core/testsrc/org/aspectj/ajdt/ajc/AjdtCommandTestCase.java +++ b/org.aspectj.ajdt.core/testsrc/org/aspectj/ajdt/ajc/AjdtCommandTestCase.java @@ -13,7 +13,7 @@ package org.aspectj.ajdt.ajc; -import org.aspectj.ajdt.internal.core.builder.AjBuildConfig; +//import org.aspectj.ajdt.internal.core.builder.AjBuildConfig; import org.aspectj.bridge.*; import org.aspectj.util.*; import org.eclipse.jdt.core.compiler.InvalidInputException; @@ -30,7 +30,7 @@ public class AjdtCommandTestCase extends TestCase { private ArrayList tempFiles = new ArrayList(); private StreamPrintWriter outputWriter = new StreamPrintWriter(new PrintWriter(System.out)); - private AjdtCommand command = new AjdtCommand(); +// private AjdtCommand command = new AjdtCommand(); private MessageWriter messageWriter = new MessageWriter(outputWriter, false); private CountingMessageHandler counter = new CountingMessageHandler(messageWriter); @@ -118,7 +118,7 @@ public class AjdtCommandTestCase extends TestCase { } public void testIncrementalOption() throws InvalidInputException { - AjBuildConfig config = command.genBuildConfig( + AjdtCommand.genBuildConfig( new String[] { "-incremental" }, counter); @@ -127,7 +127,7 @@ public class AjdtCommandTestCase extends TestCase { outputWriter.getContents().indexOf("specify a source root") != -1); outputWriter.flushBuffer(); - config = command.genBuildConfig( + AjdtCommand.genBuildConfig( new String[] { "-incremental", "-sourceroots", AjdtAjcTests.TESTDATA_PATH + "/src1" }, counter); @@ -137,7 +137,7 @@ public class AjdtCommandTestCase extends TestCase { outputWriter.getContents().equals("")); outputWriter.flushBuffer(); - config = command.genBuildConfig( + AjdtCommand.genBuildConfig( new String[] { "-incremental", "testdata/src1/Hello.java" }, counter); @@ -148,7 +148,7 @@ public class AjdtCommandTestCase extends TestCase { public void testBadOptionAndUsagePrinting() throws InvalidInputException { try { - command.genBuildConfig( + AjdtCommand.genBuildConfig( new String[] { "-mubleBadOption" }, counter); } catch (AbortException ae) { } @@ -170,7 +170,7 @@ public class AjdtCommandTestCase extends TestCase { try { try { - command.genBuildConfig( + AjdtCommand.genBuildConfig( args, counter); } catch (AbortException ae) { } @@ -195,7 +195,7 @@ public class AjdtCommandTestCase extends TestCase { try { try { - command.genBuildConfig( + AjdtCommand.genBuildConfig( args, counter); } catch (AbortException ae) { } @@ -211,7 +211,7 @@ public class AjdtCommandTestCase extends TestCase { } public void testNonExistingLstFile() { - command.genBuildConfig( + AjdtCommand.genBuildConfig( new String[] { "@mumbleDoesNotExist" }, counter); diff --git a/org.aspectj.ajdt.core/testsrc/org/aspectj/ajdt/ajc/BuildArgParserTestCase.java b/org.aspectj.ajdt.core/testsrc/org/aspectj/ajdt/ajc/BuildArgParserTestCase.java index 2259e3808..3cefb1e2c 100644 --- a/org.aspectj.ajdt.core/testsrc/org/aspectj/ajdt/ajc/BuildArgParserTestCase.java +++ b/org.aspectj.ajdt.core/testsrc/org/aspectj/ajdt/ajc/BuildArgParserTestCase.java @@ -49,7 +49,7 @@ public class BuildArgParserTestCase extends TestCase { System.setProperty("java.class.path", ENTRY); // see finally below BuildArgParser parser = new BuildArgParser(messageWriter); AjBuildConfig config = parser.genBuildConfig(new String[] { }); - String err = parser.getOtherMessages(true); + /*String err = */parser.getOtherMessages(true); //!!!assertTrue(err, null == err); assertTrue( config.getClasspath().toString(), @@ -71,7 +71,7 @@ public class BuildArgParserTestCase extends TestCase { parser = new BuildArgParser(messageWriter); config = parser.genBuildConfig(new String[] { "-1.3" }); - err = parser.getOtherMessages(true); + /*err = */parser.getOtherMessages(true); //!!!assertTrue(err, null == err); assertTrue( config.getClasspath().toString(), @@ -157,7 +157,7 @@ public class BuildArgParserTestCase extends TestCase { public void testBadPathToSourceFiles() { CountingMessageHandler countingHandler = new CountingMessageHandler(messageWriter); - AjBuildConfig config = genBuildConfig(new String[]{ "inventedDir/doesntexist/*.java"},countingHandler); + /*AjBuildConfig config = */genBuildConfig(new String[]{ "inventedDir/doesntexist/*.java"},countingHandler); assertTrue("Expected an error for the invalid path.",countingHandler.numMessages(IMessage.ERROR,false)==1); } @@ -368,7 +368,7 @@ public class BuildArgParserTestCase extends TestCase { } public void testXlint() throws InvalidInputException { - AjdtCommand command = new AjdtCommand(); +// AjdtCommand command = new AjdtCommand(); AjBuildConfig config = genBuildConfig(new String[] {"-Xlint"}, messageWriter); assertTrue("", config.getLintMode().equals(AjBuildConfig.AJLINT_DEFAULT)); config = genBuildConfig(new String[] {"-Xlint:warn"}, messageWriter); @@ -381,14 +381,14 @@ public class BuildArgParserTestCase extends TestCase { public void testXlintfile() throws InvalidInputException { String lintFile = AjdtAjcTests.TESTDATA_PATH + "/lintspec.properties"; - String badLintFile = "lint.props"; +// String badLintFile = "lint.props"; AjBuildConfig config = genBuildConfig(new String[] {"-Xlintfile", lintFile}, messageWriter); assertTrue(new File(lintFile).exists()); assertEquals(getCanonicalPath(new File(lintFile)),config.getLintSpecFile().getAbsolutePath()); } public void testOptions() throws InvalidInputException { - AjdtCommand command = new AjdtCommand(); +// AjdtCommand command = new AjdtCommand(); String TARGET = "1.4"; AjBuildConfig config = genBuildConfig(new String[] {"-target", TARGET, "-source", TARGET}, messageWriter); assertTrue( @@ -404,7 +404,7 @@ public class BuildArgParserTestCase extends TestCase { String SOURCE_PATH_1 = "A.java"; String SOURCE_PATH_2 = "B.java"; - File f = new File(FILE_PATH); +// File f = new File(FILE_PATH); AjBuildConfig config = genBuildConfig(new String[] { "@" + FILE_PATH }, messageWriter); List resultList = config.getFiles(); diff --git a/org.aspectj.ajdt.core/testsrc/org/aspectj/ajdt/internal/compiler/batch/BasicCommandTestCase.java b/org.aspectj.ajdt.core/testsrc/org/aspectj/ajdt/internal/compiler/batch/BasicCommandTestCase.java index 1e53cdd02..cd0ec9ddd 100644 --- a/org.aspectj.ajdt.core/testsrc/org/aspectj/ajdt/internal/compiler/batch/BasicCommandTestCase.java +++ b/org.aspectj.ajdt.core/testsrc/org/aspectj/ajdt/internal/compiler/batch/BasicCommandTestCase.java @@ -14,13 +14,13 @@ package org.aspectj.ajdt.internal.compiler.batch; import java.io.File; import java.util.ArrayList; -import java.util.Arrays; +//import java.util.Arrays; import java.util.List; import org.aspectj.ajdt.ajc.*; import org.aspectj.ajdt.ajc.AjdtCommand; import org.aspectj.bridge.ICommand; -import org.aspectj.bridge.IMessage; +//import org.aspectj.bridge.IMessage; import org.aspectj.bridge.MessageHandler; /** @@ -117,7 +117,7 @@ public class BasicCommandTestCase extends CommandTestCase { ICommand command = new AjdtCommand(); MessageHandler myHandler = new MessageHandler(); //myHandler.setInterceptor(org.aspectj.tools.ajc.Main.MessagePrinter.TERSE); - boolean result = command.runCommand((String[])args.toArray(new String[args.size()]), myHandler); + /*boolean result = */command.runCommand((String[])args.toArray(new String[args.size()]), myHandler); //System.err.println("messages: " + Arrays.asList(myHandler.getMessages(IMessage.INFO, true))); // DON'T yet have a way of testing that we actually got a particular info message @@ -137,7 +137,7 @@ public class BasicCommandTestCase extends CommandTestCase { ICommand command = new AjdtCommand(); MessageHandler myHandler = new MessageHandler(); myHandler.setInterceptor(org.aspectj.tools.ajc.Main.MessagePrinter.TERSE); - boolean result = command.runCommand((String[])args.toArray(new String[args.size()]), myHandler); + /*boolean result = */command.runCommand((String[])args.toArray(new String[args.size()]), myHandler); assertEquals("error for org.aspectj.lang.JoinPoint not found", 1, myHandler.getErrors().length); } diff --git a/org.aspectj.ajdt.core/testsrc/org/aspectj/ajdt/internal/compiler/batch/CommandTestCase.java b/org.aspectj.ajdt.core/testsrc/org/aspectj/ajdt/internal/compiler/batch/CommandTestCase.java index b9ab0cb30..c50ac5c89 100644 --- a/org.aspectj.ajdt.core/testsrc/org/aspectj/ajdt/internal/compiler/batch/CommandTestCase.java +++ b/org.aspectj.ajdt.core/testsrc/org/aspectj/ajdt/internal/compiler/batch/CommandTestCase.java @@ -83,7 +83,7 @@ public abstract class CommandTestCase extends TestCase { public void rerunCompiler(ICommand command) { MessageHandler myHandler = new MessageHandler(); - List recompiledFiles = new ArrayList(); +// List recompiledFiles = new ArrayList(); if (!command.repeatCommand(myHandler)) { assertTrue("recompile failed", false); } diff --git a/org.aspectj.ajdt.core/testsrc/org/aspectj/ajdt/internal/compiler/batch/IncrementalCase.java b/org.aspectj.ajdt.core/testsrc/org/aspectj/ajdt/internal/compiler/batch/IncrementalCase.java index e8cd9ab21..b612c3bbe 100644 --- a/org.aspectj.ajdt.core/testsrc/org/aspectj/ajdt/internal/compiler/batch/IncrementalCase.java +++ b/org.aspectj.ajdt.core/testsrc/org/aspectj/ajdt/internal/compiler/batch/IncrementalCase.java @@ -32,7 +32,7 @@ import java.util.ArrayList; import java.util.Arrays; import java.util.BitSet; import java.util.Collections; -import java.util.Enumeration; +//import java.util.Enumeration; import java.util.Iterator; import java.util.List; import java.util.Properties; @@ -49,7 +49,7 @@ public class IncrementalCase { // XXX NOT bound to junit - bridge tests? public static void main(String[] args) throws IOException { IncrementalCase me = new IncrementalCase(); MessageHandler h = new MessageHandler(); - boolean result; +// boolean result; StringBuffer sb = new StringBuffer(); for (int i = 0; i < args.length; i++) { sb.append("\n###### results for " + args[i]); @@ -99,7 +99,7 @@ public class IncrementalCase { // XXX NOT bound to junit - bridge tests? boolean result = true; final String toSuffix = ".java"; - final String canonicalFrom = srcBase.getCanonicalPath(); +// final String canonicalFrom = srcBase.getCanonicalPath(); final Definition[] defs = getDefinitions(srcBase); if ((null == defs) || (defs.length < 9)) { throw new Error("did not get definitions"); @@ -439,7 +439,7 @@ public class IncrementalCase { // XXX NOT bound to junit - bridge tests? String[] eclipseErrors; String[] eclipseWarnings; Definition(String prefix, Properties props) { - Enumeration keys = props.keys(); +// Enumeration keys = props.keys(); this.prefix = prefix; files = get(props, prefix + ".files"); recompiled = get(props, prefix + ".recompiled"); diff --git a/org.aspectj.ajdt.core/testsrc/org/aspectj/ajdt/internal/core/builder/AjBuildManagerTest.java b/org.aspectj.ajdt.core/testsrc/org/aspectj/ajdt/internal/core/builder/AjBuildManagerTest.java index f602539c0..5cf5d8a8c 100644 --- a/org.aspectj.ajdt.core/testsrc/org/aspectj/ajdt/internal/core/builder/AjBuildManagerTest.java +++ b/org.aspectj.ajdt.core/testsrc/org/aspectj/ajdt/internal/core/builder/AjBuildManagerTest.java @@ -13,7 +13,7 @@ package org.aspectj.ajdt.internal.core.builder; import java.io.File; -import java.io.FileOutputStream; +//import java.io.FileOutputStream; import java.io.IOException; import java.io.PrintWriter; @@ -352,11 +352,11 @@ public class AjBuildManagerTest extends TestCase { // } // } - private void touch(File file, boolean isAppend) throws IOException { - FileOutputStream s = new FileOutputStream(file.getAbsolutePath(), isAppend); - s.write(new byte[] {1,2,3}); - s.close(); - } +// private void touch(File file, boolean isAppend) throws IOException { +// FileOutputStream s = new FileOutputStream(file.getAbsolutePath(), isAppend); +// s.write(new byte[] {1,2,3}); +// s.close(); +// } /* diff --git a/taskdefs/src/org/aspectj/tools/ant/taskdefs/Ajc11CompilerAdapter.java b/taskdefs/src/org/aspectj/tools/ant/taskdefs/Ajc11CompilerAdapter.java index bd594c369..65e878628 100644 --- a/taskdefs/src/org/aspectj/tools/ant/taskdefs/Ajc11CompilerAdapter.java +++ b/taskdefs/src/org/aspectj/tools/ant/taskdefs/Ajc11CompilerAdapter.java @@ -19,9 +19,9 @@ import org.apache.tools.ant.taskdefs.compilers.CompilerAdapter; import org.aspectj.util.FileUtil; import java.io.File; -import java.io.FileFilter; -import java.io.FileWriter; -import java.io.IOException; +//import java.io.FileFilter; +//import java.io.FileWriter; +//import java.io.IOException; /** * Adapt ajc to javac commands. diff --git a/taskdefs/src/org/aspectj/tools/ant/taskdefs/Ajc2.java b/taskdefs/src/org/aspectj/tools/ant/taskdefs/Ajc2.java index 1582b1324..7e7ded5cf 100644 --- a/taskdefs/src/org/aspectj/tools/ant/taskdefs/Ajc2.java +++ b/taskdefs/src/org/aspectj/tools/ant/taskdefs/Ajc2.java @@ -490,7 +490,7 @@ public class Ajc2 extends Javac { line = line.substring(0, isharp); } - int istar = -1; +// int istar = -1; // Argument if (line.startsWith("-")) { @@ -498,7 +498,7 @@ public class Ajc2 extends Javac { } // If there are stars we'll try to resolve the file here - else if ((istar = line.indexOf("*")) != -1) { + else if (line.indexOf("*") != -1) { log("The argfile line '" + line + "' is invalid", Project.MSG_WARN); } diff --git a/taskdefs/src/org/aspectj/tools/ant/taskdefs/AjcTask.java b/taskdefs/src/org/aspectj/tools/ant/taskdefs/AjcTask.java index 798aeba31..0bf34a24e 100644 --- a/taskdefs/src/org/aspectj/tools/ant/taskdefs/AjcTask.java +++ b/taskdefs/src/org/aspectj/tools/ant/taskdefs/AjcTask.java @@ -1768,12 +1768,12 @@ public static class GuardedCommand { } } - private void addFile(File file) { - if (null != file) { - String path = file.getAbsolutePath(); - addFlag(path, true); - } - } +// private void addFile(File file) { +// if (null != file) { +// String path = file.getAbsolutePath(); +// addFlag(path, true); +// } +// } List extractArguments() { ArrayList result = new ArrayList(); -- 2.39.5