diff options
author | Paul Phillips <paulp@improving.org> | 2009-08-27 12:01:31 +0000 |
---|---|---|
committer | Paul Phillips <paulp@improving.org> | 2009-08-27 12:01:31 +0000 |
commit | de8a10cdd1badae2a73c8506a59eebaa1c323312 (patch) | |
tree | 484d35b03a5a70ce6f902a42eeaeb6f7a64ab11e /src | |
parent | b611f2e978a0b46922b9d605f2c4bf588a8024bf (diff) | |
download | scala-de8a10cdd1badae2a73c8506a59eebaa1c323312.tar.gz scala-de8a10cdd1badae2a73c8506a59eebaa1c323312.tar.bz2 scala-de8a10cdd1badae2a73c8506a59eebaa1c323312.zip |
Fix for #2240.
Diffstat (limited to 'src')
-rw-r--r-- | src/compiler/scala/tools/nsc/typechecker/RefChecks.scala | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/src/compiler/scala/tools/nsc/typechecker/RefChecks.scala b/src/compiler/scala/tools/nsc/typechecker/RefChecks.scala index 326ce6f748..42d0dd80fb 100644 --- a/src/compiler/scala/tools/nsc/typechecker/RefChecks.scala +++ b/src/compiler/scala/tools/nsc/typechecker/RefChecks.scala @@ -631,6 +631,9 @@ abstract class RefChecks extends InfoTransform { nonSensibleWarning(pre+"values of types "+normalizeAll(qual.tpe.widen)+" and "+normalizeAll(args.head.tpe.widen), alwaysEqual) // @MAT normalize for consistency in error message, otherwise part is normalized due to use of `typeSymbol', but the rest isn't def hasObjectEquals = receiver.info.member(nme.equals_) == Object_equals + // see if it has any == methods beyond Any's and AnyRef's: ticket #2240 + def hasOverloadedEqEq = receiver.info.member(nme.EQ).alternatives.length > 2 + if (formal == UnitClass && actual == UnitClass) nonSensible("", true) else if ((receiver == BooleanClass || receiver == UnitClass) && @@ -646,7 +649,7 @@ abstract class RefChecks extends InfoTransform { (name == nme.EQ || name == nme.LE)) nonSensible("non-null ", false) else if ((isNew(qual) || isNew(args.head)) && hasObjectEquals && - (name == nme.EQ || name == nme.NE)) + (name == nme.EQ || name == nme.NE) && !hasOverloadedEqEq) nonSensibleWarning("a fresh object", false) case _ => } |