aboutsummaryrefslogtreecommitdiffstats
path: root/weaver
diff options
context:
space:
mode:
authorAndy Clement <aclement@gopivotal.com>2015-05-12 12:14:55 -0700
committerAndy Clement <aclement@gopivotal.com>2015-05-12 12:14:55 -0700
commitef453a5ff86b4cf84e0d7845f7233fa1f0b3907b (patch)
treedb2aa0f30a4112a6b3535c5de621793c3e05cb34 /weaver
parente7c66d3794b07c1a44df75b5baba4ac2937c6eb8 (diff)
downloadaspectj-ef453a5ff86b4cf84e0d7845f7233fa1f0b3907b.tar.gz
aspectj-ef453a5ff86b4cf84e0d7845f7233fa1f0b3907b.zip
Do not attach INTERFACE modifier to methods from interfaces
Diffstat (limited to 'weaver')
-rw-r--r--weaver/src/org/aspectj/weaver/bcel/BcelMethod.java6
-rw-r--r--weaver/testsrc/org/aspectj/weaver/bcel/WorldTestCase.java2
2 files changed, 3 insertions, 5 deletions
diff --git a/weaver/src/org/aspectj/weaver/bcel/BcelMethod.java b/weaver/src/org/aspectj/weaver/bcel/BcelMethod.java
index 13f883f1e..7fc78b59d 100644
--- a/weaver/src/org/aspectj/weaver/bcel/BcelMethod.java
+++ b/weaver/src/org/aspectj/weaver/bcel/BcelMethod.java
@@ -83,8 +83,7 @@ class BcelMethod extends ResolvedMemberImpl {
BcelMethod(BcelObjectType declaringType, Method method) {
super(method.getName().equals("<init>") ? CONSTRUCTOR : (method.getName().equals("<clinit>") ? STATIC_INITIALIZATION
- : METHOD), declaringType.getResolvedTypeX(), declaringType.isInterface() ? method.getModifiers()
- | Modifier.INTERFACE : method.getModifiers(), method.getName(), method.getSignature());
+ : METHOD), declaringType.getResolvedTypeX(), method.getModifiers(), method.getName(), method.getSignature());
this.method = method;
sourceContext = declaringType.getResolvedTypeX().getSourceContext();
bcelObjectType = declaringType;
@@ -97,8 +96,7 @@ class BcelMethod extends ResolvedMemberImpl {
*/
BcelMethod(BcelObjectType declaringType, Method method, List<AjAttribute> attributes) {
super(method.getName().equals("<init>") ? CONSTRUCTOR : (method.getName().equals("<clinit>") ? STATIC_INITIALIZATION
- : METHOD), declaringType.getResolvedTypeX(), declaringType.isInterface() ? method.getModifiers()
- | Modifier.INTERFACE : method.getModifiers(), method.getName(), method.getSignature());
+ : METHOD), declaringType.getResolvedTypeX(), method.getModifiers(), method.getName(), method.getSignature());
this.method = method;
sourceContext = declaringType.getResolvedTypeX().getSourceContext();
bcelObjectType = declaringType;
diff --git a/weaver/testsrc/org/aspectj/weaver/bcel/WorldTestCase.java b/weaver/testsrc/org/aspectj/weaver/bcel/WorldTestCase.java
index 111d7e54d..1fb756852 100644
--- a/weaver/testsrc/org/aspectj/weaver/bcel/WorldTestCase.java
+++ b/weaver/testsrc/org/aspectj/weaver/bcel/WorldTestCase.java
@@ -123,7 +123,7 @@ public class WorldTestCase extends CommonWorldTests {
});
ResolvedMember remove = iter.lookupMethod(MemberImpl.method(iter, 0, "remove", "()V"));
assertNotNull("iterator doesn't have remove", remove);
- modifiersTest(remove, Modifier.PUBLIC | Modifier.INTERFACE); // no longer abstract in Java8 (default instead)
+ modifiersTest(remove, Modifier.PUBLIC); // no longer abstract in Java8 (default instead)
exceptionsTest(remove, UnresolvedType.NONE);
ResolvedMember clone = iter.lookupMethod(MemberImpl.method(UnresolvedType.OBJECT, 0, "clone", "()Ljava/lang/Object;"));