Merge pull request #11788 from dotty-staging/fix-11781

Move classType checks for new expressions to PostTyper

Author: liufengyun

compiler/src/dotty/tools/dotc/transform/PostTyper.scala

@@ -295,6 +295,7 @@ class PostTyper extends MacroTransform with IdentityDenotTransformer { thisPhase
             case Select(nu: New, nme.CONSTRUCTOR) if isCheckable(nu) =>
               // need to check instantiability here, because the type of the New itself
               // might be a type constructor.
+              ctx.typer.checkClassType(tree.tpe, tree.srcPos, traitReq = false, stablePrefixReq = true)
               Checking.checkInstantiable(tree.tpe, nu.srcPos)
               withNoCheckNews(nu :: Nil)(app1)
             case _ =>

compiler/src/dotty/tools/dotc/typer/Typer.scala

@@ -772,10 +772,6 @@ class Typer extends Namer
       case _ =>
         var tpt1 = typedType(tree.tpt)
         tpt1 = tpt1.withType(ensureAccessible(tpt1.tpe, superAccess = false, tpt1.srcPos))
-
-        if (checkClassType(typeOfNew(tpt1), tpt1.srcPos, traitReq = false, stablePrefixReq = true) eq defn.ObjectType)
-          tpt1 = TypeTree(defn.ObjectType).withSpan(tpt1.span)
-
         tpt1 match {
           case AppliedTypeTree(_, targs) =>
             for case targ: TypeBoundsTree <- targs do

tests/neg-custom-args/fatal-warnings/pureStatement.scala

@@ -26,7 +26,7 @@ object Test {
 
   doSideEffects(1) // error: pure expression does nothing in statement position
 
-  val broken = new IDontExist("") // error // error
+  val broken = new IDontExist("") // error
   broken.foo // no extra error, and no pure expression warning
   broken.foo() // same
 }

tests/neg/alloc-abstract.scala

@@ -3,13 +3,13 @@ class Test[T] {
 
   type Foo[T] = Array[T]
 
-  new T                   // error: not a class type
-  new T()                 // error: not a class type
-  new U                   // error: not a class type
-  new U()                 // error: not a class type
-  new IArray[String]      // error: not a class type
-  new IArray[String]()    // error: not a class type
-  new IArray[String](10)  // error: not a class type // error: too mamy arguments
+  new T                   // error: does not have a constructor
+  new T()                 // error: does not have a constructor
+  new U                   // error: does not have a constructor
+  new U()                 // error: does not have a constructor
+  new IArray[String]      // error: does not have a constructor
+  new IArray[String]()    // error: does not have a constructor
+  new IArray[String](10)  // error: does not have a constructor
 
   new Foo[String](10)     // ok
 }

tests/neg/creator-applys.scala

@@ -10,26 +10,8 @@ class Test {
     def run = s"C $x $y"
   }
 
-  val x1 = new Test().A()      // error: object A does not take parameters
-  val x2 = new Test().B()      // error: value B is not a member of Test
-  val x3 = new Test().B[Int]() // error: value B is not a member of Test
+  val x1 = new Test().A()      // error: not stable
+  val x2 = new Test().B()      // error: not stable
+  val x3 = new Test().B[Int]() // error: not stable
 }
 
-
-object Test2 {
-  class A(s: String = "A") {
-    def run = s
-  }
-  object A {
-    def apply() = A("X")  // error: recursive method needs return type
-  }
-}
-
-object Test3 {
-  class A(s: String = "A") {
-    def run = s
-  }
-  object A {
-    def apply(): A = A("X")  // error too many arguments
-  }
-}

tests/neg/creator-applys2.scala

@@ -0,0 +1,18 @@
+
+object Test2 {
+  class A(s: String = "A") {
+    def run = s
+  }
+  object A {
+    def apply() = A("X")  // error: recursive method needs return type
+  }
+}
+
+object Test3 {
+  class A(s: String = "A") {
+    def run = s
+  }
+  object A {
+    def apply(): A = A("X")  // error too many arguments
+  }
+}

tests/neg/i11781.scala

@@ -0,0 +1,4 @@
+
+trait A:
+  type Foo
+  new Foo  // error: does not have a constructor

tests/neg/i8569.check

@@ -1,12 +1,6 @@
--- [E083] Type Error: tests/neg/i8569.scala:8:8 ------------------------------------------------------------------------
-8 |  outer.Inner(2) // error
-  |  ^^^^^^^^^^^
-  |  Outer is not a valid class prefix, since it is not an immutable path
-
-longer explanation available when compiling with `-explain`
--- [E083] Type Error: tests/neg/i8569.scala:9:6 ------------------------------------------------------------------------
-9 |  new outer.Inner(2) // error
-  |      ^^^^^
-  |      (Test.outer : => Outer) is not a valid type prefix, since it is not an immutable path
+-- [E083] Type Error: tests/neg/i8569.scala:13:21 ----------------------------------------------------------------------
+13 |  val x = outer.Inner(2) // error (at posttyper)
+   |          ^^^^^^^^^^^^^^
+   |          Outer is not a valid class prefix, since it is not an immutable path
 
 longer explanation available when compiling with `-explain`

tests/neg/i8569.scala

@@ -2,9 +2,13 @@ class Outer(x: Int) {
   class Inner(y: Int) {
   }
 }
+class Outer2(x: Int) {
+  class Inner(y: Int) {
+  }
+}
 object Test {
   def outer = Outer(1)
+  def outer2 = Outer2(1)
 
-  outer.Inner(2) // error
-  new outer.Inner(2) // error
+  val x = outer.Inner(2) // error (at posttyper)
 }

tests/neg/i8569a.check

@@ -0,0 +1,6 @@
+-- [E083] Type Error: tests/neg/i8569a.scala:13:14 ---------------------------------------------------------------------
+13 |  val x = new outer2.Inner(2) // error (at typer)
+   |              ^^^^^^
+   |              (Test.outer2 : => Outer2) is not a valid type prefix, since it is not an immutable path
+
+longer explanation available when compiling with `-explain`