Skip to content

Commit ec87be9

Browse files
committed
Don't report synthetic erased bridge methods (e.g. apply) #36
The correctly typed apply method is already reported so don't report the erased method as well.
1 parent 25b6ed2 commit ec87be9

File tree

2 files changed

+2
-3
lines changed
  • reporter

2 files changed

+2
-3
lines changed
Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
11
class A was concrete; is declared abstract in new version
2-
method apply()java.lang.Object in object A does not have a correspondent in new version
32
method apply()A in object A does not have a correspondent in new version
43
the type hierarchy of object A has changed in new version. Missing types {scala.runtime.AbstractFunction0}

reporter/src/main/scala/com/typesafe/tools/mima/lib/analyze/method/MethodChecker.scala

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ private[analyze] abstract class BaseMethodChecker extends Checker[MemberInfo, Cl
3737

3838
private[analyze] class ClassMethodChecker extends BaseMethodChecker {
3939
def check(method: MemberInfo, inclazz: ClassInfo): Option[Problem] = {
40-
if (method.nonBridge && method.nonAccessible)
40+
if (method.nonAccessible)
4141
None
4242
else if (method.isDeferred)
4343
super.check(method, inclazz.lookupMethods(method.bytecodeName))
@@ -48,7 +48,7 @@ private[analyze] class ClassMethodChecker extends BaseMethodChecker {
4848

4949
private[analyze] class TraitMethodChecker extends BaseMethodChecker {
5050
def check(method: MemberInfo, inclazz: ClassInfo): Option[Problem] = {
51-
if (method.nonBridge && method.nonAccessible)
51+
if (method.nonAccessible)
5252
None
5353
else if (method.owner.hasStaticImpl(method))
5454
checkStaticImplMethod(method, inclazz)

0 commit comments

Comments
 (0)