diff options
author | Adriaan Moors <adriaan@lightbend.com> | 2016-12-05 08:48:36 +0100 |
---|---|---|
committer | Adriaan Moors <adriaan@lightbend.com> | 2016-12-05 08:48:55 +0100 |
commit | 7bf8ffa155ba66311a904c7eeaca79a70aa7e6f7 (patch) | |
tree | 774ea0d46857cbb3939a270c5e8915f2a3964c1d | |
parent | 7a57c6eec6c37e8ca3a7f182f0cf2604d7bc80df (diff) | |
download | scala-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.scala | 2 | ||||
-rw-r--r-- | test/files/run/junitForwarders/C_1.scala | 4 |
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$ - ") } |