fixed 1275 by adding minimal early check to Nam...

fixed 1275 by adding minimal early check to Namers so that overriding of type members in refinements cannot change number of type parameters (in principle the full overriding checks should be performed at a later point, when they don't cause cyclicity errors -- this is TODO)

1 files changed, 18 insertions, 1 deletions

diff --git a/src/compiler/scala/tools/nsc/typechecker/Namers.scala b/src/compiler/scala/tools/nsc/typechecker/Namers.scala
index 2e5ab49907..e82252c2ee 100644
--- a/src/compiler/scala/tools/nsc/typechecker/Namers.scala
+++ b/src/compiler/scala/tools/nsc/typechecker/Namers.scala
@@ -671,7 +671,24 @@ trait Namers { self: Analyzer =>
           TypeBounds(AllClass.tpe, AnyClass.tpe)
         case tp => tp
       }
-      parameterizedType(tparamSyms, tp) //@M
+
+      def verifyOverriding(other: Symbol): Boolean = {
+        if(other.unsafeTypeParams.length != tparamSyms.length) {
+          context.error(tpsym.pos,
+              "The kind of "+tpsym.keyString+" "+tpsym.varianceString + tpsym.nameString+
+              " does not conform to the expected kind of " + other.defString + other.locationString + ".")
+          false
+        } else true
+      }
+
+      // @M: make sure overriding in refinements respects rudimentary kinding
+      // have to do this early, as otherwise we might get crashes: (see neg/bug1275.scala)
+      //   suppose some parameterized type member is overridden by a type member w/o params,
+      //   then appliedType will be called on a type that does not expect type args --> crash
+      if (tpsym.owner.isRefinementClass &&  // only needed in refinements
+          !tpsym.allOverriddenSymbols.forall{verifyOverriding(_)})
+	      ErrorType
+      else parameterizedType(tparamSyms, tp)
     }
 
     def typeSig(tree: Tree): Type = {