aboutsummaryrefslogtreecommitdiffstats
path: root/weaver
diff options
context:
space:
mode:
Diffstat (limited to 'weaver')
-rw-r--r--weaver/src/org/aspectj/weaver/Advice.java43
-rw-r--r--weaver/src/org/aspectj/weaver/AjAttribute.java118
-rw-r--r--weaver/src/org/aspectj/weaver/IntMap.java2
-rw-r--r--weaver/src/org/aspectj/weaver/ResolvedMember.java21
-rw-r--r--weaver/src/org/aspectj/weaver/ResolvedPointcutDefinition.java2
-rw-r--r--weaver/src/org/aspectj/weaver/TypeX.java2
-rw-r--r--weaver/src/org/aspectj/weaver/World.java15
-rw-r--r--weaver/src/org/aspectj/weaver/bcel/BcelAdvice.java31
-rw-r--r--weaver/src/org/aspectj/weaver/bcel/BcelWorld.java12
-rw-r--r--weaver/src/org/aspectj/weaver/patterns/CflowPointcut.java2
-rw-r--r--weaver/src/org/aspectj/weaver/patterns/WildTypePattern.java4
-rw-r--r--weaver/src/org/aspectj/weaver/patterns/WithinPointcut.java4
-rw-r--r--weaver/testdata/dummyAspect.jarbin589 -> 589 bytes
-rw-r--r--weaver/testdata/megatrace.jarbin4858 -> 4895 bytes
-rw-r--r--weaver/testdata/megatraceNoweave.jarbin2649 -> 2696 bytes
-rw-r--r--weaver/testdata/tracing.jarbin2213 -> 2268 bytes
-rw-r--r--weaver/testsrc/org/aspectj/weaver/bcel/PointcutResidueTestCase.java4
17 files changed, 183 insertions, 77 deletions
diff --git a/weaver/src/org/aspectj/weaver/Advice.java b/weaver/src/org/aspectj/weaver/Advice.java
index e903b6c42..b6714333f 100644
--- a/weaver/src/org/aspectj/weaver/Advice.java
+++ b/weaver/src/org/aspectj/weaver/Advice.java
@@ -21,10 +21,10 @@ import org.aspectj.weaver.patterns.*;
public abstract class Advice extends ShadowMunger {
- protected AdviceKind kind;
+ AjAttribute.AdviceAttribute attribute; // the pointcut field is ignored
+
+ protected AdviceKind kind; // alias of attribute.getKind()
protected Member signature;
- protected int extraParameterFlags;
- protected int lexicalPosition;
// not necessarily declaring aspect, this is a semantics change from 1.0
protected ResolvedTypeX concreteAspect; // null until after concretize
@@ -73,14 +73,12 @@ public abstract class Advice extends ShadowMunger {
}
- public Advice(AdviceKind kind, Pointcut pointcut, Member signature,
- int extraParameterFlags, int start, int end, ISourceContext sourceContext)
+ public Advice(AjAttribute.AdviceAttribute attribute, Pointcut pointcut, Member signature)
{
- super(pointcut, start, end, sourceContext);
- this.kind = kind;
+ super(pointcut, attribute.getStart(), attribute.getEnd(), attribute.getSourceContext());
+ this.attribute = attribute;
+ this.kind = attribute.getKind(); // alias
this.signature = signature;
- this.extraParameterFlags = extraParameterFlags;
- this.lexicalPosition = start; //XXX should go away
}
@@ -143,11 +141,15 @@ public abstract class Advice extends ShadowMunger {
}
public boolean hasExtraParameter() {
- return (extraParameterFlags & ExtraArgument) != 0;
+ return (getExtraParameterFlags() & ExtraArgument) != 0;
+ }
+
+ protected int getExtraParameterFlags() {
+ return attribute.getExtraParameterFlags();
}
protected int getExtraParameterCount() {
- return countOnes(extraParameterFlags & ParameterMask);
+ return countOnes(getExtraParameterFlags() & ParameterMask);
}
public static int countOnes(int bits) {
@@ -160,7 +162,7 @@ public abstract class Advice extends ShadowMunger {
}
public int getBaseParameterCount() {
- return signature.getParameterTypes().length - getExtraParameterCount();
+ return getSignature().getParameterTypes().length - getExtraParameterCount();
}
public TypeX getExtraParameterType() {
@@ -173,10 +175,10 @@ public abstract class Advice extends ShadowMunger {
}
protected String extraParametersToString() {
- if (extraParameterFlags == 0) {
+ if (getExtraParameterFlags() == 0) {
return "";
} else {
- return "(extraFlags: " + extraParameterFlags + ")";
+ return "(extraFlags: " + getExtraParameterFlags() + ")";
}
}
@@ -197,7 +199,7 @@ public abstract class Advice extends ShadowMunger {
p.state = Pointcut.CONCRETE;
}
- Advice munger = world.concreteAdvice(kind, p, signature, extraParameterFlags, start, end, sourceContext);
+ Advice munger = world.concreteAdvice(attribute, p, signature);
munger.concreteAspect = fromType;
//System.err.println("concretizing here " + p + " with clause " + clause);
return munger;
@@ -218,8 +220,9 @@ public abstract class Advice extends ShadowMunger {
public boolean equals(Object other) {
if (! (other instanceof Advice)) return false;
Advice o = (Advice) other;
- return o.kind == kind && o.pointcut.equals(pointcut) && o.signature.equals(signature) &&
- o.extraParameterFlags == extraParameterFlags;
+ return o.attribute.equals(attribute)
+ && o.pointcut.equals(pointcut)
+ && o.signature.equals(signature);
}
private volatile int hashCode = 0;
public int hashCode() {
@@ -243,9 +246,11 @@ public abstract class Advice extends ShadowMunger {
public static final int ParameterMask = 0xf;
public static final int CanInline = 0x40;
-
+
+
+ // for testing only
public void setLexicalPosition(int lexicalPosition) {
- this.lexicalPosition = lexicalPosition;
+ start = lexicalPosition;
}
public ResolvedTypeX getConcreteAspect() {
diff --git a/weaver/src/org/aspectj/weaver/AjAttribute.java b/weaver/src/org/aspectj/weaver/AjAttribute.java
index 864513371..85e0195a4 100644
--- a/weaver/src/org/aspectj/weaver/AjAttribute.java
+++ b/weaver/src/org/aspectj/weaver/AjAttribute.java
@@ -185,7 +185,7 @@ public abstract class AjAttribute {
}
public void write(DataOutputStream s) throws IOException {
s.writeUTF(sourceFileName);
- FileUtil.writeIntArray(s, lineBreaks);
+ FileUtil.writeIntArray(lineBreaks, s);
}
public static SourceContextAttribute read(DataInputStream s) throws IOException {
@@ -249,51 +249,101 @@ public abstract class AjAttribute {
private AdviceKind kind;
private Pointcut pointcut;
- private int extraArgumentFlags;
+ private int extraParameterFlags;
private int start;
private int end;
private ISourceContext sourceContext;
+ // these are only used by around advice
+ private boolean proceedInInners;
+ private ResolvedMember[] proceedCallSignatures; // size == # of proceed calls in body
+ private boolean[] formalsUnchangedToProceed; // size == formals.size
+ private TypeX[] declaredExceptions;
+
/**
* @param lexicalPosition must be greater than the lexicalPosition
* of any advice declared before this one in an aspect, otherwise,
* it can be any value.
*/
- public AdviceAttribute(AdviceKind kind, Pointcut pointcut, int extraArgumentFlags, int start, int end, ISourceContext sourceContext) {
+ public AdviceAttribute(AdviceKind kind, Pointcut pointcut, int extraArgumentFlags,
+ int start, int end, ISourceContext sourceContext) {
+ this.kind = kind;
+ this.pointcut = pointcut;
+ this.extraParameterFlags = extraArgumentFlags;
+ this.start = start;
+ this.end = end;
+ this.sourceContext = sourceContext;
+
+ //XXX put this back when testing works better (or fails better)
+ //if (kind == AdviceKind.Around) throw new IllegalArgumentException("not for around");
+ }
+
+ public AdviceAttribute(AdviceKind kind, Pointcut pointcut, int extraArgumentFlags,
+ int start, int end, ISourceContext sourceContext,
+ boolean proceedInInners, ResolvedMember[] proceedCallSignatures,
+ boolean[] formalsUnchangedToProceed, TypeX[] declaredExceptions) {
this.kind = kind;
this.pointcut = pointcut;
- this.extraArgumentFlags = extraArgumentFlags;
+ this.extraParameterFlags = extraArgumentFlags;
this.start = start;
this.end = end;
this.sourceContext = sourceContext;
+
+ if (kind != AdviceKind.Around) throw new IllegalArgumentException("only for around");
+
+ this.proceedInInners = proceedInInners;
+ this.proceedCallSignatures = proceedCallSignatures;
+ this.formalsUnchangedToProceed = formalsUnchangedToProceed;
+ this.declaredExceptions = declaredExceptions;
}
public static AdviceAttribute read(DataInputStream s, ISourceContext context) throws IOException {
- return new AdviceAttribute(
- AdviceKind.read(s),
- Pointcut.read(s, context),
- s.readByte(),
- s.readInt(), s.readInt(), context);
+ AdviceKind kind = AdviceKind.read(s);
+ if (kind == AdviceKind.Around) {
+ return new AdviceAttribute(
+ kind,
+ Pointcut.read(s, context),
+ s.readByte(),
+ s.readInt(), s.readInt(), context,
+ s.readBoolean(),
+ ResolvedMember.readResolvedMemberArray(s, context),
+ FileUtil.readBooleanArray(s),
+ TypeX.readArray(s));
+ } else {
+ return new AdviceAttribute(
+ kind,
+ Pointcut.read(s, context),
+ s.readByte(),
+ s.readInt(), s.readInt(), context);
+ }
}
public void write(DataOutputStream s) throws IOException {
kind.write(s);
pointcut.write(s);
- s.writeByte(extraArgumentFlags);
+ s.writeByte(extraParameterFlags);
s.writeInt(start);
s.writeInt(end);
+
+ if (kind == AdviceKind.Around) {
+ s.writeBoolean(proceedInInners);
+ ResolvedMember.writeArray(proceedCallSignatures, s);
+ FileUtil.writeBooleanArray(formalsUnchangedToProceed, s);
+ TypeX.writeArray(declaredExceptions, s);
+ }
}
+
public Advice reify(Member signature, World world) {
- return world.concreteAdvice(kind, pointcut, signature, extraArgumentFlags, start, end, sourceContext);
+ return world.concreteAdvice(this, pointcut, signature);
}
public String toString() {
return "AdviceAttribute(" + kind + ", " + pointcut + ", " +
- extraArgumentFlags + ", " + start+")";
+ extraParameterFlags + ", " + start+")";
}
- public int getExtraArgumentFlags() {
- return extraArgumentFlags;
+ public int getExtraParameterFlags() {
+ return extraParameterFlags;
}
public AdviceKind getKind() {
@@ -304,6 +354,34 @@ public abstract class AjAttribute {
return pointcut;
}
+ public TypeX[] getDeclaredExceptions() {
+ return declaredExceptions;
+ }
+
+ public boolean[] getFormalsUnchangedToProceed() {
+ return formalsUnchangedToProceed;
+ }
+
+ public ResolvedMember[] getProceedCallSignatures() {
+ return proceedCallSignatures;
+ }
+
+ public boolean isProceedInInners() {
+ return proceedInInners;
+ }
+
+ public int getEnd() {
+ return end;
+ }
+
+ public ISourceContext getSourceContext() {
+ return sourceContext;
+ }
+
+ public int getStart() {
+ return start;
+ }
+
}
public static class Aspect extends AjAttribute {
@@ -339,10 +417,7 @@ public abstract class AjAttribute {
this.accessedMembers = accessedMembers;
}
public void write(DataOutputStream s) throws IOException {
- s.writeInt(accessedMembers.length);
- for (int i = 0, len = accessedMembers.length; i < len; i++) {
- accessedMembers[i].write(s);
- }
+ ResolvedMember.writeArray(accessedMembers, s);
}
public ResolvedMember[] getAccessedMembers() {
@@ -350,12 +425,7 @@ public abstract class AjAttribute {
}
public static PrivilegedAttribute read(DataInputStream s, ISourceContext context) throws IOException {
- int len = s.readInt();
- ResolvedMember[] members = new ResolvedMember[len];
- for (int i=0; i < len; i++) {
- members[i] = ResolvedMember.readResolvedMember(s, context);
- }
- return new PrivilegedAttribute(members);
+ return new PrivilegedAttribute(ResolvedMember.readResolvedMemberArray(s, context));
}
}
diff --git a/weaver/src/org/aspectj/weaver/IntMap.java b/weaver/src/org/aspectj/weaver/IntMap.java
index 7a12bf0f6..a1b24d382 100644
--- a/weaver/src/org/aspectj/weaver/IntMap.java
+++ b/weaver/src/org/aspectj/weaver/IntMap.java
@@ -54,7 +54,7 @@ public class IntMap {
}
public Member getAdviceSignature() {
- if (enclosingAdvice instanceof Advice) return ((Advice)enclosingAdvice).signature;
+ if (enclosingAdvice instanceof Advice) return ((Advice)enclosingAdvice).getSignature();
else return null;
}
diff --git a/weaver/src/org/aspectj/weaver/ResolvedMember.java b/weaver/src/org/aspectj/weaver/ResolvedMember.java
index 4851f9174..6469ee5f2 100644
--- a/weaver/src/org/aspectj/weaver/ResolvedMember.java
+++ b/weaver/src/org/aspectj/weaver/ResolvedMember.java
@@ -92,12 +92,20 @@ public class ResolvedMember extends Member implements IHasPosition {
s.writeInt(modifiers);
s.writeUTF(getName());
s.writeUTF(getSignature());
- TypeX.write(getExceptions(), s);
+ TypeX.writeArray(getExceptions(), s);
s.writeInt(getStart());
s.writeInt(getEnd());
}
+
+ public static void writeArray(ResolvedMember[] members, DataOutputStream s) throws IOException {
+ s.writeInt(members.length);
+ for (int i = 0, len = members.length; i < len; i++) {
+ members[i].write(s);
+ }
+ }
+
public static ResolvedMember readResolvedMember(DataInputStream s, ISourceContext sourceContext) throws IOException {
ResolvedMember m = new ResolvedMember(Kind.read(s), TypeX.read(s), s.readInt(), s.readUTF(), s.readUTF());
@@ -108,6 +116,17 @@ public class ResolvedMember extends Member implements IHasPosition {
return m;
}
+ public static ResolvedMember[] readResolvedMemberArray(DataInputStream s, ISourceContext context) throws IOException {
+ int len = s.readInt();
+ ResolvedMember[] members = new ResolvedMember[len];
+ for (int i=0; i < len; i++) {
+ members[i] = ResolvedMember.readResolvedMember(s, context);
+ }
+ return members;
+ }
+
+
+
public ResolvedMember resolve(World world) {
return this;
}
diff --git a/weaver/src/org/aspectj/weaver/ResolvedPointcutDefinition.java b/weaver/src/org/aspectj/weaver/ResolvedPointcutDefinition.java
index 5dfc055d8..49cec0f61 100644
--- a/weaver/src/org/aspectj/weaver/ResolvedPointcutDefinition.java
+++ b/weaver/src/org/aspectj/weaver/ResolvedPointcutDefinition.java
@@ -46,7 +46,7 @@ public class ResolvedPointcutDefinition extends ResolvedMember {
getDeclaringType().write(s);
s.writeInt(getModifiers());
s.writeUTF(getName());
- TypeX.write(getParameterTypes(), s);
+ TypeX.writeArray(getParameterTypes(), s);
pointcut.write(s);
}
diff --git a/weaver/src/org/aspectj/weaver/TypeX.java b/weaver/src/org/aspectj/weaver/TypeX.java
index 7d5c46fd4..42cc3b09c 100644
--- a/weaver/src/org/aspectj/weaver/TypeX.java
+++ b/weaver/src/org/aspectj/weaver/TypeX.java
@@ -599,7 +599,7 @@ public class TypeX {
}
}
- public static void write(TypeX[] types, DataOutputStream s) throws IOException {
+ public static void writeArray(TypeX[] types, DataOutputStream s) throws IOException {
int len = types.length;
s.writeShort(len);
for (int i=0; i < len; i++) {
diff --git a/weaver/src/org/aspectj/weaver/World.java b/weaver/src/org/aspectj/weaver/World.java
index d31b9e7a4..4ed8abb49 100644
--- a/weaver/src/org/aspectj/weaver/World.java
+++ b/weaver/src/org/aspectj/weaver/World.java
@@ -177,8 +177,7 @@ public abstract class World {
public ResolvedTypeX resolveObjectType(TypeX ty) {
return ResolvedTypeX.MISSING;
}
- public Advice concreteAdvice(AdviceKind kind, Pointcut p, Member m, int extraMods,
- int start, int end, ISourceContext context) {
+ public Advice concreteAdvice(AjAttribute.AdviceAttribute attribute, Pointcut p, Member m) {
throw new RuntimeException("unimplemented");
}
public ConcreteTypeMunger concreteTypeMunger(ResolvedTypeMunger munger, ResolvedTypeX aspectType) {
@@ -188,11 +187,9 @@ public abstract class World {
public abstract Advice concreteAdvice(
- AdviceKind kind,
- Pointcut p,
- Member signature,
- int extraParameterFlags,
- int start, int end, ISourceContext context);
+ AjAttribute.AdviceAttribute attribute,
+ Pointcut pointcut,
+ Member signature);
public final Advice concreteAdvice(
AdviceKind kind,
@@ -201,7 +198,9 @@ public abstract class World {
int extraParameterFlags,
IHasSourceLocation loc)
{
- return concreteAdvice(kind, p, signature, extraParameterFlags, loc.getStart(), loc.getEnd(), loc.getSourceContext());
+ AjAttribute.AdviceAttribute attribute =
+ new AjAttribute.AdviceAttribute(kind, p, extraParameterFlags, loc.getStart(), loc.getEnd(), loc.getSourceContext());
+ return concreteAdvice(attribute, p, signature);
}
diff --git a/weaver/src/org/aspectj/weaver/bcel/BcelAdvice.java b/weaver/src/org/aspectj/weaver/bcel/BcelAdvice.java
index e6ff472ee..e27102cfa 100644
--- a/weaver/src/org/aspectj/weaver/bcel/BcelAdvice.java
+++ b/weaver/src/org/aspectj/weaver/bcel/BcelAdvice.java
@@ -28,12 +28,23 @@ public class BcelAdvice extends Advice {
private Test pointcutTest;
private ExposedState exposedState;
+ public BcelAdvice(
+ AjAttribute.AdviceAttribute attribute,
+ Pointcut pointcut,
+ Member signature,
+ ResolvedTypeX concreteAspect)
+ {
+ super(attribute, pointcut, signature);
+ this.concreteAspect = concreteAspect;
+ }
+
+ // !!! only used for testing
public BcelAdvice(AdviceKind kind, Pointcut pointcut, Member signature,
int extraArgumentFlags,
int start, int end, ISourceContext sourceContext, ResolvedTypeX concreteAspect)
{
- super(kind, pointcut, signature, extraArgumentFlags, start, end, sourceContext);
- this.concreteAspect = concreteAspect;
+ this(new AjAttribute.AdviceAttribute(kind, pointcut, extraArgumentFlags, start, end, sourceContext),
+ pointcut, signature, concreteAspect);
}
// ---- implementations of ShadowMunger's methods
@@ -61,15 +72,15 @@ public class BcelAdvice extends Advice {
pointcutTest = getPointcut().findResidue(shadow, exposedState);
// make sure thisJoinPoint parameters are initialized
- if ((extraParameterFlags & ThisJoinPointStaticPart) != 0) {
+ if ((getExtraParameterFlags() & ThisJoinPointStaticPart) != 0) {
((BcelShadow)shadow).getThisJoinPointStaticPartVar();
}
- if ((extraParameterFlags & ThisJoinPoint) != 0) {
+ if ((getExtraParameterFlags() & ThisJoinPoint) != 0) {
((BcelShadow)shadow).getThisJoinPointVar();
}
- if ((extraParameterFlags & ThisEnclosingJoinPointStaticPart) != 0) {
+ if ((getExtraParameterFlags() & ThisEnclosingJoinPointStaticPart) != 0) {
((BcelShadow)shadow).getThisEnclosingJoinPointStaticPartVar();
}
}
@@ -214,16 +225,16 @@ public class BcelAdvice extends Advice {
// handle thisJoinPoint parameters
// these need to be in that same order as parameters in
// org.aspectj.ajdt.internal.compiler.ast.AdviceDeclaration
- if ((extraParameterFlags & ThisJoinPointStaticPart) != 0) {
+ if ((getExtraParameterFlags() & ThisJoinPointStaticPart) != 0) {
shadow.getThisJoinPointStaticPartBcelVar().appendLoad(il, fact);
}
- if ((extraParameterFlags & ThisJoinPoint) != 0) {
+ if ((getExtraParameterFlags() & ThisJoinPoint) != 0) {
shadow.getThisJoinPointBcelVar().appendLoad(il, fact);
}
- if ((extraParameterFlags & ThisEnclosingJoinPointStaticPart) != 0) {
+ if ((getExtraParameterFlags() & ThisEnclosingJoinPointStaticPart) != 0) {
shadow.getThisEnclosingJoinPointStaticPartBcelVar().appendLoad(il, fact);
}
@@ -289,9 +300,9 @@ public class BcelAdvice extends Advice {
if (declaringAspect == o_declaringAspect) {
if (kind.isAfter() || o.kind.isAfter()) {
- return this.lexicalPosition < o.lexicalPosition ? -1: +1;
+ return this.getStart() < o.getStart() ? -1: +1;
} else {
- return this.lexicalPosition < o.lexicalPosition ? +1: -1;
+ return this.getStart()< o.getStart() ? +1: -1;
}
} else if (declaringAspect.isAssignableFrom(o_declaringAspect)) {
return -1;
diff --git a/weaver/src/org/aspectj/weaver/bcel/BcelWorld.java b/weaver/src/org/aspectj/weaver/bcel/BcelWorld.java
index 1ff0c4e34..6b6987e16 100644
--- a/weaver/src/org/aspectj/weaver/bcel/BcelWorld.java
+++ b/weaver/src/org/aspectj/weaver/bcel/BcelWorld.java
@@ -106,7 +106,7 @@ public class BcelWorld extends World {
Pointcut p =
pointcut.resolve(new SimpleScope(this, bindings));
- return concreteAdvice(kind, p, m, extraFlag, 0, 0, null);
+ return new BcelAdvice(kind, p, m, extraFlag, 0, 0, null, null);
}
private String[] parseIds(String str) {
@@ -297,14 +297,12 @@ public class BcelWorld extends World {
}
public Advice concreteAdvice(
- AdviceKind kind,
- Pointcut p,
- Member signature,
- int extraParameterFlags,
- int start, int end, ISourceContext sourceContext)
+ AjAttribute.AdviceAttribute attribute,
+ Pointcut pointcut,
+ Member signature)
{
//System.err.println("concrete advice: " + signature + " context " + sourceContext);
- return new BcelAdvice(kind, p, signature, extraParameterFlags, start, end, sourceContext, null);
+ return new BcelAdvice(attribute, pointcut, signature, null);
}
public ConcreteTypeMunger concreteTypeMunger(
diff --git a/weaver/src/org/aspectj/weaver/patterns/CflowPointcut.java b/weaver/src/org/aspectj/weaver/patterns/CflowPointcut.java
index 39566938d..7eeb58629 100644
--- a/weaver/src/org/aspectj/weaver/patterns/CflowPointcut.java
+++ b/weaver/src/org/aspectj/weaver/patterns/CflowPointcut.java
@@ -55,7 +55,7 @@ public class CflowPointcut extends Pointcut {
s.writeByte(Pointcut.CFLOW);
entry.write(s);
s.writeBoolean(isBelow);
- FileUtil.writeIntArray(s, freeVars);
+ FileUtil.writeIntArray(freeVars, s);
writeLocation(s);
}
public static Pointcut read(DataInputStream s, ISourceContext context) throws IOException {
diff --git a/weaver/src/org/aspectj/weaver/patterns/WildTypePattern.java b/weaver/src/org/aspectj/weaver/patterns/WildTypePattern.java
index c12c0e8dc..4eef87f66 100644
--- a/weaver/src/org/aspectj/weaver/patterns/WildTypePattern.java
+++ b/weaver/src/org/aspectj/weaver/patterns/WildTypePattern.java
@@ -433,8 +433,8 @@ public class WildTypePattern extends TypePattern {
s.writeInt(dim);
//??? storing this information with every type pattern is wasteful of .class
// file size. Storing it on enclosing types would be more efficient
- FileUtil.writeStringArray(s, knownMatches);
- FileUtil.writeStringArray(s, importedPrefixes);
+ FileUtil.writeStringArray(knownMatches, s);
+ FileUtil.writeStringArray(importedPrefixes, s);
writeLocation(s);
}
diff --git a/weaver/src/org/aspectj/weaver/patterns/WithinPointcut.java b/weaver/src/org/aspectj/weaver/patterns/WithinPointcut.java
index 7e8663ca2..81c6bfcd4 100644
--- a/weaver/src/org/aspectj/weaver/patterns/WithinPointcut.java
+++ b/weaver/src/org/aspectj/weaver/patterns/WithinPointcut.java
@@ -28,6 +28,10 @@ public class WithinPointcut extends Pointcut {
public FuzzyBoolean match(Shadow shadow) {
TypeX enclosingType = shadow.getEnclosingType();
+// if (shadow.getKind() == Shadow.FieldSet) {
+// System.err.println("within?" + type + " matches " + enclosingType + " on " + shadow);
+// }
+
while (enclosingType != null) {
if (type.matchesStatically(shadow.getIWorld().resolve(enclosingType))) {
return FuzzyBoolean.YES;
diff --git a/weaver/testdata/dummyAspect.jar b/weaver/testdata/dummyAspect.jar
index 5cadbea53..784a3b611 100644
--- a/weaver/testdata/dummyAspect.jar
+++ b/weaver/testdata/dummyAspect.jar
Binary files differ
diff --git a/weaver/testdata/megatrace.jar b/weaver/testdata/megatrace.jar
index e227960cc..eda290200 100644
--- a/weaver/testdata/megatrace.jar
+++ b/weaver/testdata/megatrace.jar
Binary files differ
diff --git a/weaver/testdata/megatraceNoweave.jar b/weaver/testdata/megatraceNoweave.jar
index 86df6e86d..fcf480940 100644
--- a/weaver/testdata/megatraceNoweave.jar
+++ b/weaver/testdata/megatraceNoweave.jar
Binary files differ
diff --git a/weaver/testdata/tracing.jar b/weaver/testdata/tracing.jar
index ed778dde5..c3f520133 100644
--- a/weaver/testdata/tracing.jar
+++ b/weaver/testdata/tracing.jar
Binary files differ
diff --git a/weaver/testsrc/org/aspectj/weaver/bcel/PointcutResidueTestCase.java b/weaver/testsrc/org/aspectj/weaver/bcel/PointcutResidueTestCase.java
index 7dafd2f9b..e2fd9dd87 100644
--- a/weaver/testsrc/org/aspectj/weaver/bcel/PointcutResidueTestCase.java
+++ b/weaver/testsrc/org/aspectj/weaver/bcel/PointcutResidueTestCase.java
@@ -183,7 +183,7 @@ public class PointcutResidueTestCase extends WeaveTestCase {
));
ShadowMunger pp =
- world.concreteAdvice(
+ new BcelAdvice(
AdviceKind.Before,
rp,
Member.method(
@@ -193,7 +193,7 @@ public class PointcutResidueTestCase extends WeaveTestCase {
Member.typesToSignature(
ResolvedTypeX.VOID,
TypeX.forNames(formalTypes))),
- 0, -1, -1, null);
+ 0, -1, -1, null, null);
ResolvedTypeX inAspect = world.resolve("Aspect");
CrosscuttingMembers xcut = new CrosscuttingMembers(inAspect);
generated by cgit v1.2.3 (git 2.39.1) at 2025-01-08 10:36:54 +0000