summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAdriaan Moors <adriaan@lightbend.com>2016-12-05 08:48:36 +0100
committerAdriaan Moors <adriaan@lightbend.com>2016-12-05 08:48:55 +0100
commit7bf8ffa155ba66311a904c7eeaca79a70aa7e6f7 (patch)
tree774ea0d46857cbb3939a270c5e8915f2a3964c1d
parent7a57c6eec6c37e8ca3a7f182f0cf2604d7bc80df (diff)
downloadscala-7bf8ffa155ba66311a904c7eeaca79a70aa7e6f7.tar.gz
scala-7bf8ffa155ba66311a904c7eeaca79a70aa7e6f7.tar.bz2
scala-7bf8ffa155ba66311a904c7eeaca79a70aa7e6f7.zip
Drop annotations from trait static super accessor
Based on review suggestion by retronym. See also scala/scala-dev#213
-rw-r--r--src/compiler/scala/tools/nsc/backend/jvm/BCodeSkelBuilder.scala2
-rw-r--r--test/files/run/junitForwarders/C_1.scala4
2 files changed, 3 insertions, 3 deletions
diff --git a/src/compiler/scala/tools/nsc/backend/jvm/BCodeSkelBuilder.scala b/src/compiler/scala/tools/nsc/backend/jvm/BCodeSkelBuilder.scala
index fdb5687311..03df1c76fa 100644
--- a/src/compiler/scala/tools/nsc/backend/jvm/BCodeSkelBuilder.scala
+++ b/src/compiler/scala/tools/nsc/backend/jvm/BCodeSkelBuilder.scala
@@ -494,7 +494,7 @@ abstract class BCodeSkelBuilder extends BCodeHelpers {
genDefDef(statified)
} else {
val forwarderDefDef = {
- val dd1 = global.gen.mkStatic(deriveDefDef(dd)(_ => EmptyTree), traitSuperAccessorName(sym), _.cloneSymbol)
+ val dd1 = global.gen.mkStatic(deriveDefDef(dd)(_ => EmptyTree), traitSuperAccessorName(sym), _.cloneSymbol.withoutAnnotations)
dd1.symbol.setFlag(Flags.ARTIFACT).resetFlag(Flags.OVERRIDE)
val selfParam :: realParams = dd1.vparamss.head.map(_.symbol)
deriveDefDef(dd1)(_ =>
diff --git a/test/files/run/junitForwarders/C_1.scala b/test/files/run/junitForwarders/C_1.scala
index 2af2026a61..0361ef42ef 100644
--- a/test/files/run/junitForwarders/C_1.scala
+++ b/test/files/run/junitForwarders/C_1.scala
@@ -10,6 +10,6 @@ object Test extends App {
assert(s == e, s"found: $s\nexpected: $e")
}
check(classOf[C], "foo - @org.junit.Test()")
- // TODO scala-dev#213: should `foo$` really carry the @Test annotation?
- check(classOf[T], "$init$ - ;foo - @org.junit.Test();foo$ - @org.junit.Test()")
+ // scala/scala-dev#213, scala/scala#5570: `foo$` should not have the @Test annotation
+ check(classOf[T], "$init$ - ;foo - @org.junit.Test();foo$ - ")
}