Make check look intentional per review

Author: som-snytt

compiler/src/dotty/tools/dotc/transform/localopt/FormatChecker.scala

@@ -228,22 +228,20 @@ class TypedFormatChecker(partsElems: List[Tree], parts: List[String], args: List
       case _ => true
 
     def lintToString(arg: Type): Unit =
-      def check(tp: Type): Boolean = tp.widen match
+      def checkIsStringify(tp: Type): Boolean = tp.widen match
         case OrType(tp1, tp2) =>
-          check(tp1) || check(tp2)
+          checkIsStringify(tp1) || checkIsStringify(tp2)
         case tp =>
-          if tp =:= defn.StringType then
-            false
-          else if tp =:= defn.UnitType then
-            warningAt(CC)("interpolated Unit value")
-            true
-          else if !tp.isPrimitiveValueType then
-            warningAt(CC)("interpolation uses toString")
-            true
-          else
-            false
-      if ctx.settings.Whas.toStringInterpolated && kind == StringXn && check(arg) then
-        ()
+          !(tp =:= defn.StringType)
+          && {
+              tp =:= defn.UnitType
+              && { warningAt(CC)("interpolated Unit value"); true }
+            ||
+              !tp.isPrimitiveValueType
+              && { warningAt(CC)("interpolation uses toString"); true }
+          }
+      if ctx.settings.Whas.toStringInterpolated && kind == StringXn then
+        checkIsStringify(arg): Unit
 
     // what arg type if any does the conversion accept
     def acceptableVariants: List[Type] =

compiler/src/dotty/tools/dotc/transform/localopt/StringInterpolatorOpt.scala

@@ -110,22 +110,20 @@ class StringInterpolatorOpt extends MiniPhase:
       result
     end mkConcat
     def lintToString(t: Tree): Unit =
-      def check(tp: Type): Boolean = tp.widen match
+      def checkIsStringify(tp: Type): Boolean = tp.widen match
         case OrType(tp1, tp2) =>
-          check(tp1) || check(tp2)
+          checkIsStringify(tp1) || checkIsStringify(tp2)
         case tp =>
-          if tp =:= defn.StringType then
-            false
-          else if tp =:= defn.UnitType then
-            report.warning("interpolated Unit value", t.srcPos)
-            true
-          else if !tp.isPrimitiveValueType then
-            report.warning("interpolation uses toString", t.srcPos)
-            true
-          else
-            false
-      if ctx.settings.Whas.toStringInterpolated && check(t.tpe) then
-        ()
+          !(tp =:= defn.StringType)
+          && {
+              tp =:= defn.UnitType
+              && { report.warning("interpolated Unit value", t.srcPos); true }
+            ||
+              !tp.isPrimitiveValueType
+              && { report.warning("interpolation uses toString", t.srcPos); true }
+          }
+      if ctx.settings.Whas.toStringInterpolated then
+        checkIsStringify(t.tpe): Unit
     val sym = tree.symbol
     // Test names first to avoid loading scala.StringContext if not used, and common names first
     val isInterpolatedMethod =