Fix scala#9408: Warn on implicit view resolved with -source:3.0-migration

Author: prolativ

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

@@ -174,9 +174,11 @@ object Implicits:
                 migrateTo3
                 && tpw.derivesFrom(defn.FunctionClass(1))
                 && ref.symbol != defn.Predef_conforms
+
               val isImplicitConversion = tpw.derivesFrom(defn.ConversionClass)
               // An implementation of <:< counts as a view
               val isConforms = tpw.derivesFrom(defn.SubTypeClass)
+
               val conversionKind =
                 if (isFunctionInS2 || isImplicitConversion || isConforms) Candidate.Conversion
                 else Candidate.None
@@ -779,7 +781,26 @@ trait Implicits:
           SelectionProto(name, memberProto, compat, privateOK = false)
         case tp => tp
       }
-      try inferImplicit(adjust(to), from, from.span)
+      try
+        val inferred = inferImplicit(adjust(to), from, from.span)
+
+        def isOldStyleFunctionConversion(tpe: Type): Boolean =
+          tpe match {
+            case PolyType(_, resType) =>
+              isOldStyleFunctionConversion(resType)
+            case _ => tpe.derivesFrom(defn.FunctionClass(1)) && !tpe.derivesFrom(defn.ConversionClass) && !tpe.derivesFrom(defn.SubTypeClass)
+          }
+
+        inferred match {
+          case SearchSuccess(_, ref, _) =>
+            if isOldStyleFunctionConversion(ref.underlying) then
+              report.migrationWarning(
+                i"The conversion ${ref} will not be applied implicitly here in Scala 3 because only implicit methods and instances of Conversion class will continue to work as implicit views.",
+                from
+              )
+          case _ =>
+        }
+        inferred
       catch {
         case ex: AssertionError =>
           implicits.println(s"view $from ==> $to")

tests/neg-custom-args/fatal-warnings/i9408a.check

@@ -0,0 +1,24 @@
+-- Error: tests/neg-custom-args/fatal-warnings/i9408a.scala:16:20 ------------------------------------------------------
+16 |  val length: Int = "qwerty" // error
+   |                    ^^^^^^^^
+   |The conversion (Test3.implicitLength : String => Int) will not be applied implicitly here in Scala 3 because only implicit methods and instances of Conversion class will continue to work as implicit views.
+-- Error: tests/neg-custom-args/fatal-warnings/i9408a.scala:21:20 ------------------------------------------------------
+21 |  val length: Int = "qwerty" // error
+   |                    ^^^^^^^^
+   |The conversion (Test3a.implicitLength : => String => Int) will not be applied implicitly here in Scala 3 because only implicit methods and instances of Conversion class will continue to work as implicit views.
+-- Error: tests/neg-custom-args/fatal-warnings/i9408a.scala:26:20 ------------------------------------------------------
+26 |  val length: Int = "qwerty" // error
+   |                    ^^^^^^^^
+   |The conversion (Test3b.implicitLength : [A] => String => Int) will not be applied implicitly here in Scala 3 because only implicit methods and instances of Conversion class will continue to work as implicit views.
+-- Error: tests/neg-custom-args/fatal-warnings/i9408a.scala:31:20 ------------------------------------------------------
+31 |  val length: Int = "qwerty" // error
+   |                    ^^^^^^^^
+   |The conversion (Test4.implicitLength : Map[String, Int]) will not be applied implicitly here in Scala 3 because only implicit methods and instances of Conversion class will continue to work as implicit views.
+-- Error: tests/neg-custom-args/fatal-warnings/i9408a.scala:35:60 ------------------------------------------------------
+35 |  implicit def a2int[A](a: A)(implicit ev: A => Int): Int = a // error
+   |                                                            ^
+   |The conversion (ev : A => Int) will not be applied implicitly here in Scala 3 because only implicit methods and instances of Conversion class will continue to work as implicit views.
+-- Error: tests/neg-custom-args/fatal-warnings/i9408a.scala:55:2 -------------------------------------------------------
+55 |  123.foo // error
+   |  ^^^
+   |The conversion (Test8.a2foo : [A] => A => Test8.Foo) will not be applied implicitly here in Scala 3 because only implicit methods and instances of Conversion class will continue to work as implicit views.

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

@@ -0,0 +1,63 @@
+import language.`3.0-migration`
+import scala.language.implicitConversions
+
+object Test1 {
+  implicit def implicitLength(s: String): Int = s.length
+  val length: Int = "qwerty" // ok
+}
+
+object Test2 {
+  implicit val implicitLength: Conversion[String, Int] = _.length
+  val length: Int = "qwerty" // ok
+}
+
+object Test3 {
+  implicit val implicitLength: String => Int = _.length
+  val length: Int = "qwerty" // error
+}
+
+object Test3a {
+  implicit def implicitLength: String => Int = _.length
+  val length: Int = "qwerty" // error
+}
+
+object Test3b {
+  implicit def implicitLength[A]: String => Int = _.length
+  val length: Int = "qwerty" // error
+}
+
+object Test4 {
+  implicit val implicitLength: Map[String, Int] = Map("qwerty" -> 6)
+  val length: Int = "qwerty" // error
+}
+
+object Test5 {
+  implicit def a2int[A](a: A)(implicit ev: A => Int): Int = a // error
+}
+
+object Test6 {
+  implicit def a2int[A](a: A)(implicit ev: A => Int): Int = ev(a) // ok
+}
+
+object Test7 {
+  trait Foo {
+    def foo = "foo"
+  }
+  implicit def a2foo[A](a: A): Foo = new Foo {}
+  123.foo // ok
+}
+
+object Test8 {
+  trait Foo {
+    def foo = "foo"
+  }
+  implicit def a2foo[A]: A => Foo = _ => new Foo {}
+  123.foo // error
+}
+
+object Test10 {
+  implicit class FooOps(a: Any) {
+    def foo = "foo"
+  }
+  123.foo // ok
+}

tests/neg-custom-args/fatal-warnings/i9408b.check

@@ -0,0 +1,4 @@
+-- Error: tests/neg-custom-args/fatal-warnings/i9408b/Test.scala:7:20 --------------------------------------------------
+7 |  val length: Int = "abc" // error
+  |                    ^^^^^
+  |The conversion (test.conversions.Conv.implicitLength : String => Int) will not be applied implicitly here in Scala 3 because only implicit methods and instances of Conversion class will continue to work as implicit views.

tests/neg-custom-args/fatal-warnings/i9408b/Conv.scala

@@ -0,0 +1,8 @@
+package test.conversions
+
+import language.`3.0-migration`
+import scala.language.implicitConversions
+
+object Conv {
+  implicit val implicitLength: String => Int = _.length
+}

tests/neg-custom-args/fatal-warnings/i9408b/Test.scala

@@ -0,0 +1,8 @@
+import language.`3.0-migration`
+import scala.language.implicitConversions
+
+
+object Test {
+  import test.conversions.Conv._
+  val length: Int = "abc" // error
+}