Skip to content

Commit 8123de4

Browse files
committed
Fix #9166: Harden check for values in patterns based on class inheritance info
1 parent 758782a commit 8123de4

File tree

2 files changed

+25
-4
lines changed

2 files changed

+25
-4
lines changed

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

Lines changed: 19 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3808,10 +3808,25 @@ class Typer extends Namer
38083808
mapOver(tp)
38093809
}
38103810

3811-
if tree.symbol.isOneOf(Module | Enum)
3812-
&& !(tree.tpe frozen_<:< pt) // fast track
3813-
&& !(tree.tpe frozen_<:< approx(pt))
3814-
then
3811+
val sym = tree.tpe.widen.classSymbol
3812+
3813+
// Is it certain that a value of `tree.tpe` is never a subtype of `pt`?
3814+
// It is true if either
3815+
// - the class of `tree.tpe` and class of `pt` cannot have common subclass, or
3816+
// - `tree` is an object or enum value, which cannot possibly be a subtype of `pt`
3817+
val isDefiniteNotSubtype = {
3818+
val clsA = tree.tpe.widenDealias.classSymbol
3819+
val clsB = pt.dealias.classSymbol
3820+
clsA.exists && clsB.exists
3821+
&& !clsA.isNullableClass
3822+
&& (!clsA.isNumericValueClass || !clsB.isNumericValueClass)
3823+
&& !clsA.asClass.mayHaveCommonChild(clsB.asClass)
3824+
|| tree.symbol.isOneOf(Module | Enum)
3825+
&& !(tree.tpe frozen_<:< pt) // fast track
3826+
&& !(tree.tpe frozen_<:< approx(pt))
3827+
}
3828+
3829+
if isDefiniteNotSubtype then
38153830
// We could check whether `equals` is overriden.
38163831
// Reasons for not doing so:
38173832
// - it complicates the protocol

tests/neg/i9166.scala

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
object UnitTest extends App {
2+
def foo(m: Unit) = m match {
3+
case runtime.BoxedUnit.UNIT => println("ok") // error
4+
}
5+
foo(())
6+
}

0 commit comments

Comments
 (0)