diff options
author | aclement <aclement> | 2011-09-07 22:56:49 +0000 |
---|---|---|
committer | aclement <aclement> | 2011-09-07 22:56:49 +0000 |
commit | e3ec5bdcf25e6a3e9385c856085575365873af2a (patch) | |
tree | 8dd9107332654bb9ca2256753e793cd0d07c7299 /weaver | |
parent | 476b222ee32dbb61177a58cc764d405c89b6db4f (diff) | |
download | aspectj-e3ec5bdcf25e6a3e9385c856085575365873af2a.tar.gz aspectj-e3ec5bdcf25e6a3e9385c856085575365873af2a.zip |
357019: message to lint
Diffstat (limited to 'weaver')
-rw-r--r-- | weaver/src/org/aspectj/weaver/bcel/BcelClassWeaver.java | 14 |
1 files changed, 10 insertions, 4 deletions
diff --git a/weaver/src/org/aspectj/weaver/bcel/BcelClassWeaver.java b/weaver/src/org/aspectj/weaver/bcel/BcelClassWeaver.java index 8e6f0deff..0cfc4cf08 100644 --- a/weaver/src/org/aspectj/weaver/bcel/BcelClassWeaver.java +++ b/weaver/src/org/aspectj/weaver/bcel/BcelClassWeaver.java @@ -80,7 +80,6 @@ import org.aspectj.weaver.Shadow; import org.aspectj.weaver.ShadowMunger; import org.aspectj.weaver.UnresolvedType; import org.aspectj.weaver.UnresolvedTypeVariableReferenceType; -import org.aspectj.weaver.WeaverMessages; import org.aspectj.weaver.WeaverStateInfo; import org.aspectj.weaver.World; import org.aspectj.weaver.model.AsmRelationshipProvider; @@ -402,8 +401,15 @@ class BcelClassWeaver implements IClassWeaver { */ public boolean weave() { if (clazz.isWoven() && !clazz.isReweavable()) { - world.showMessage(IMessage.ERROR, WeaverMessages.format(WeaverMessages.ALREADY_WOVEN, clazz.getType().getName()), - ty.getSourceLocation(), null); + if (world.getLint().nonReweavableTypeEncountered.isEnabled()) { + world.getLint().nonReweavableTypeEncountered.signal(clazz.getType().getName(), ty.getSourceLocation()); + } + // Integer uniqueID = new Integer(rm.hashCode() * deca.hashCode()); + // if (!reportedProblems.contains(uniqueID)) { + // reportedProblems.add(uniqueID); + // world.getLint().elementAlreadyAnnotated.signal(new String[] { rm.toString(), + // world.showMessage(IMessage.ERROR, WeaverMessages.format(WeaverMessages.ALREADY_WOVEN, clazz.getType().getName()), + // ty.getSourceLocation(), null); return false; } @@ -3103,7 +3109,7 @@ class BcelClassWeaver implements IClassWeaver { && s.charAt(4) == 'a' && (s.equals("org.aspectj.runtime.internal.CFlowCounter") || s.equals("org.aspectj.runtime.internal.CFlowStack") || s - .equals("org.aspectj.runtime.reflect.Factory"))) { + .equals("org.aspectj.runtime.reflect.Factory"))) { proceed = false; } else { if (methodName.equals("aspectOf")) { |