Skip to content

Commit ec5cbcc

Browse files
committed
Do not crash when typing a closure with unknown type, since it can occur for erroneous input
1 parent de2d35c commit ec5cbcc

File tree

4 files changed

+16
-3
lines changed

4 files changed

+16
-3
lines changed

Diff for: compiler/src/dotty/tools/dotc/parsing/Parsers.scala

+1-1
Original file line numberDiff line numberDiff line change
@@ -1725,7 +1725,7 @@ object Parsers {
17251725
case arg =>
17261726
arg
17271727
val args1 = args.mapConserve(sanitize)
1728-
1728+
17291729
if in.isArrow || isPureArrow || erasedArgs.contains(true) then
17301730
functionRest(args)
17311731
else

Diff for: compiler/src/dotty/tools/dotc/typer/Typer.scala

-2
Original file line numberDiff line numberDiff line change
@@ -1977,8 +1977,6 @@ class Typer(@constructorOnly nestingLevel: Int = 0) extends Namer
19771977
// Polymorphic SAMs are not currently supported (#6904).
19781978
EmptyTree
19791979
case tp =>
1980-
if !tp.isErroneous then
1981-
throw new java.lang.Error(i"internal error: closing over non-method $tp, pos = ${tree.span}")
19821980
TypeTree(defn.AnyType)
19831981
}
19841982
else typed(tree.tpt)

Diff for: tests/neg/i20511-1.scala

+7
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
package pakiet
2+
3+
def toppingPrice(size: Int): Double = ???
4+
5+
def crustPrice(crustType: Double): Double = ???
6+
7+
export toppingPrice.apply, crustPrice.unlift // error // error // error

Diff for: tests/neg/i20511.scala

+8
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
package pakiet
2+
3+
def toppingPrice(size: Int): Double = ???
4+
5+
def crustPrice(crustType: Double): Double = ???
6+
7+
export toppingPrice, crustPrice // error // error
8+
val i = 1 // error

0 commit comments

Comments
 (0)