Deprecate +(x: String): String on primitives

This is consistent with the deprecation of any2stringadd in scala#6315.

Author: smarter

project/GenerateAnyVals.scala

@@ -146,7 +146,9 @@ import scala.language.implicitConversions"""
 
     def mkCoercions = numeric map (x => "def to%s: %s".format(x, x))
     def mkUnaryOps  = unaryOps map (x => "%s\n  def unary_%s : %s".format(x.doc, x.op, this opType I))
-    def mkStringOps = List("def +(x: String): String")
+    def mkStringOps = List(
+      """@deprecated("Adding a number and a String is deprecated. Convert the number to a String with `toString` first to call +", "2.13.0") def +(x: String): String"""
+    )
     def mkShiftOps  = (
       for (op <- shiftOps ; arg <- List(I, L)) yield
         "%s\n  def %s(x: %s): %s".format(op.doc, op.op, arg, this opType I)

src/library/scala/Byte.scala

@@ -42,7 +42,7 @@ final abstract class Byte private extends AnyVal {
   /** Returns the negation of this value. */
   def unary_- : Int
 
-  def +(x: String): String
+  @deprecated("Adding a number and a String is deprecated. Convert the number to a String with `toString` first to call +", "2.13.0") def +(x: String): String
 
   /**
   * Returns this value bit-shifted left by the specified number of bits,

src/library/scala/Char.scala

@@ -42,7 +42,7 @@ final abstract class Char private extends AnyVal {
   /** Returns the negation of this value. */
   def unary_- : Int
 
-  def +(x: String): String
+  @deprecated("Adding a number and a String is deprecated. Convert the number to a String with `toString` first to call +", "2.13.0") def +(x: String): String
 
   /**
   * Returns this value bit-shifted left by the specified number of bits,

src/library/scala/Double.scala

@@ -33,7 +33,7 @@ final abstract class Double private extends AnyVal {
   /** Returns the negation of this value. */
   def unary_- : Double
 
-  def +(x: String): String
+  @deprecated("Adding a number and a String is deprecated. Convert the number to a String with `toString` first to call +", "2.13.0") def +(x: String): String
 
   /** Returns `true` if this value is equal to x, `false` otherwise. */
   def ==(x: Byte): Boolean

src/library/scala/Float.scala

@@ -33,7 +33,7 @@ final abstract class Float private extends AnyVal {
   /** Returns the negation of this value. */
   def unary_- : Float
 
-  def +(x: String): String
+  @deprecated("Adding a number and a String is deprecated. Convert the number to a String with `toString` first to call +", "2.13.0") def +(x: String): String
 
   /** Returns `true` if this value is equal to x, `false` otherwise. */
   def ==(x: Byte): Boolean

src/library/scala/Int.scala

@@ -42,7 +42,7 @@ final abstract class Int private extends AnyVal {
   /** Returns the negation of this value. */
   def unary_- : Int
 
-  def +(x: String): String
+  @deprecated("Adding a number and a String is deprecated. Convert the number to a String with `toString` first to call +", "2.13.0") def +(x: String): String
 
   /**
   * Returns this value bit-shifted left by the specified number of bits,

src/library/scala/Long.scala

@@ -42,7 +42,7 @@ final abstract class Long private extends AnyVal {
   /** Returns the negation of this value. */
   def unary_- : Long
 
-  def +(x: String): String
+  @deprecated("Adding a number and a String is deprecated. Convert the number to a String with `toString` first to call +", "2.13.0") def +(x: String): String
 
   /**
   * Returns this value bit-shifted left by the specified number of bits,

src/library/scala/Short.scala

@@ -42,7 +42,7 @@ final abstract class Short private extends AnyVal {
   /** Returns the negation of this value. */
   def unary_- : Int
 
-  def +(x: String): String
+  @deprecated("Adding a number and a String is deprecated. Convert the number to a String with `toString` first to call +", "2.13.0") def +(x: String): String
 
   /**
   * Returns this value bit-shifted left by the specified number of bits,

test/files/jvm/duration-tck.scala

@@ -151,7 +151,7 @@ object Test extends App {
     intercept[IllegalArgumentException] { mdur / 0.9 }
     intercept[IllegalArgumentException] { dur / 0.4 }
     intercept[IllegalArgumentException] { mdur / 0.4 }
-    Duration(x + unit.toString.toLowerCase)
+    Duration(x.toString + unit.toString.toLowerCase)
     Duration("-" + x + unit.toString.toLowerCase)
     intercept[IllegalArgumentException] { Duration("%.0f".format(x + 10000000d) + unit.toString.toLowerCase) }
     intercept[IllegalArgumentException] { Duration("-%.0f".format(x + 10000000d) + unit.toString.toLowerCase) }

test/files/jvm/sync-var.scala

@@ -44,7 +44,7 @@ summers foreach { _.join() }
 
 val got = sum.get
 val expected = (n + 1) * n / 2
-println(got + " " + expected + " " + (got == expected))
+println(got.toString + " " + expected + " " + (got == expected))
 
 producers foreach { _.join() }
 

test/files/neg/numeric-add-string-warning.check

@@ -0,0 +1,6 @@
+numeric-add-string-warning.scala:2: warning: method + in class Int is deprecated (since 2.13.0): Adding a number and a String is deprecated. Convert the number to a String with `toString` first to call +
+  val x = 4 + "2"
+            ^
+error: No warnings can be incurred under -Xfatal-warnings.
+one warning found
+one error found

test/files/neg/numeric-add-string-warning.flags

@@ -0,0 +1 @@
+-Xfatal-warnings -deprecation

test/files/neg/numeric-add-string-warning.scala

@@ -0,0 +1,3 @@
+object Test {
+  val x = 4 + "2"
+}

test/files/run/Course-2002-03.scala

@@ -15,7 +15,7 @@ object M0 {
       r.denom * s.denom);
 
   def makeString(r: Rational) =
-    r.numer + "/" + r.denom;
+    s"${r.numer}/${r.denom}"
 
   val x = new Rational(1, 2);
   val y = new Rational(1, 3);
@@ -40,7 +40,7 @@ object M1 {
       new Rational(
         numer * r.numer,
         denom * r.denom);
-    override def toString() = numer + "/" + denom;
+    override def toString() = s"$numer/$denom"
   }
 
   val x = new Rational(1, 3);
@@ -77,7 +77,7 @@ object M2 {
       new Rational(
         numer * r.denom,
         denom * r.numer);
-    override def toString() = numer + "/" + denom;
+    override def toString() = s"$numer/$denom"
   }
 
   val x = new Rational(1, 3);
@@ -100,7 +100,7 @@ object M3 {
     def less(that: Rational) =
       this.numer * that.denom < that.numer * this.denom;
     def max(that: Rational) = if (this.less(that)) that else this;
-    override def toString() = numer + "/" + denom;
+    override def toString() = s"$numer/$denom"
   }
 
   val x = new Rational(66, 42);
@@ -136,7 +136,7 @@ object M4 {
       new Rational(
         numer * r.denom,
         denom * r.numer);
-    override def toString() = numer + "/" + denom;
+    override def toString() = s"$numer/$denom"
   }
 
   val x = new Rational(1, 2);

test/files/run/Course-2002-06.scala

@@ -122,7 +122,7 @@ class PostScript (filename: String, _width: Double, _height: Double)
   }
 
   def plotLine(x1: Double, y1: Double, x2: Double, y2: Double): Unit = {
-    Console.println(round(x1) + " " + round(y1) + " m " +
+    Console.println(round(x1).toString + " " + round(y1) + " m " +
                     round(x2) + " " + round(y2) + " l");
   }
 
@@ -132,7 +132,7 @@ class PostScript (filename: String, _width: Double, _height: Double)
   Console.println("%%BoundingBox: 0 0 " + mm2ps(psWidth) + " " + mm2ps(psHeight));
   Console.println("%%EndComments\n");
   Console.println("/m {moveto} bind def\n/l {lineto} bind def\n");
-  Console.println(mm2ps(line_thickness) + " setlinewidth\nnewpath");
+  Console.println(mm2ps(line_thickness).toString + " setlinewidth\nnewpath");
 
   /** Terminate the PS document and close the file stream. */
   def close : Unit = {

test/files/run/Course-2002-08.scala

@@ -135,7 +135,7 @@ object M3 {
 object M4 {
 
   def test = {
-    for (i <- range(1, 4)) { Console.print(i + " ") };
+    for (i <- range(1, 4)) { Console.print(s"$i ") };
     Console.println;
     Console.println(for (i <- range(1, 4)) yield i);
     Console.println;
@@ -295,7 +295,7 @@ object M5 {
         ain setSignal (if (a == 0) false else true);
         bin setSignal (if (b == 0) false else true);
         run;
-        Console.println(a + " & " + b + " = " + result);
+        Console.println(a.toString + " & " + b + " = " + result);
         Console.println;
       }
 
@@ -320,7 +320,7 @@ object M5 {
         ain setSignal (if (a == 0) false else true);
         bin setSignal (if (b == 0) false else true);
         run;
-        Console.println(a + " | " + b + " = " + result);
+        Console.println(a.toString + " | " + b + " = " + result);
         Console.println;
       }
 
@@ -348,7 +348,7 @@ object M5 {
         ain setSignal (if (a == 0) false else true);
         bin setSignal (if (b == 0) false else true);
         run;
-        Console.println(a + " + " + b + " = " + result);
+        Console.println(a.toString + " + " + b + " = " + result);
         Console.println;
       }
 
@@ -379,7 +379,7 @@ object M5 {
         bin setSignal (if (b == 0) false else true);
         cin setSignal (if (c == 0) false else true);
         run;
-        Console.println(a + " + " + b + " + " + c + " = " + result);
+        Console.println(a.toString + " + " + b + " + " + c + " = " + result);
         Console.println;
       }
 

test/files/run/bridges.scala

@@ -7116,7 +7116,7 @@ object Test {
 
     if (errors > 0) {
       Console.println;
-      Console.println(errors + " error" + (if (errors > 1) "s" else ""));
+      Console.println(s"$errors error" + (if (errors > 1) "s" else ""));
     }
   }
 }

test/files/run/bugs.scala

@@ -481,7 +481,7 @@ object Test  {
 
     if (errors > 0) {
       Console.println;
-      Console.println(errors + " error" + (if (errors > 1) "s" else ""));
+      Console.println(s"$errors error" + (if (errors > 1) "s" else ""));
     }
   }
 }

test/files/run/fors.scala

@@ -18,29 +18,29 @@ object Test extends App {
     println("\ntestOld")
 
     // lists
-    for (x <- xs) print(x + " "); println
+    for (x <- xs) print(s"$x "); println
     for (x <- xs;
-         if x % 2 == 0) print(x + " "); println
+         if x % 2 == 0) print(s"$x "); println
     for {x <- xs
-         if x % 2 == 0} print(x + " "); println
+         if x % 2 == 0} print(s"$x "); println
     var n = 0
     for (_ <- xs) n += 1; println(n)
-    for ((x, y) <- xs zip ys) print(x + " "); println
-    for (p @ (x, y) <- xs zip ys) print(p._1 + " "); println
+    for ((x, y) <- xs zip ys) print(s"$x "); println
+    for (p @ (x, y) <- xs zip ys) print(s"${p._1} "); println
 
     // iterators
-    for (x <- it) print(x + " "); println
+    for (x <- it) print(s"$x "); println
     for (x <- it;
-         if x % 2 == 0) print(x + " "); println
+         if x % 2 == 0) print(s"$x "); println
     for {x <- it
-         if x % 2 == 0} print(x + " "); println
+         if x % 2 == 0} print(s"$x "); println
 
     // arrays
-    for (x <- ar) print(x + " "); println
+    for (x <- ar) print(s"$x "); println
     for (x <- ar;
-         if x.toInt > 97) print(x + " "); println
+         if x.toInt > 97) print(s"$x "); println
     for {x <- ar
-         if x.toInt > 97} print(x + " "); println
+         if x.toInt > 97} print(s"$x "); println
 
   }
 
@@ -52,28 +52,28 @@ object Test extends App {
     // lists
     var n = 0
     for (_ <- xs) n += 1; println(n)
-    for ((x, y) <- xs zip ys) print(x + " "); println
-    for (p @ (x, y) <- xs zip ys) print(p._1 + " "); println
+    for ((x, y) <- xs zip ys) print(s"$x "); println
+    for (p @ (x, y) <- xs zip ys) print(s"${p._1} "); println
 
     // iterators
-    for (x <- it) print(x + " "); println
-    for (x <- it if x % 2 == 0) print(x + " "); println
-    for (x <- it; if x % 2 == 0) print(x + " "); println
+    for (x <- it) print(s"$x "); println
+    for (x <- it if x % 2 == 0) print(s"$x "); println
+    for (x <- it; if x % 2 == 0) print(s"$x "); println
     for (x <- it;
-         if x % 2 == 0) print(x + " "); println
+         if x % 2 == 0) print(s"$x "); println
     for (x <- it
-         if x % 2 == 0) print(x + " "); println
+         if x % 2 == 0) print(s"$x "); println
     for {x <- it
-         if x % 2 == 0} print(x + " "); println
+         if x % 2 == 0} print(s"$x "); println
     for (x <- it;
          y = 2
-         if x % y == 0) print(x + " "); println
+         if x % y == 0) print(s"$x "); println
     for {x <- it
          y = 2
-         if x % y == 0} print(x + " "); println
+         if x % y == 0} print(s"$x "); println
 
     // arrays
-    for (x <- ar) print(x + " "); println
+    for (x <- ar) print(s"$x "); println
 
   }
 

test/files/run/kmpSliceSearch.scala

@@ -54,7 +54,7 @@ object Test {
     )
     (haystacks zip needles) foreach {
       case (hay, nee) =>
-        println(hay.indexOfSlice(nee,2) + " " + hay.lastIndexOfSlice(nee,13))
+        println(s"${hay.indexOfSlice(nee,2)} ${hay.lastIndexOfSlice(nee,13)}")
     }
   }
 }

test/files/run/macro-auto-duplicate/Macros_1.scala

@@ -8,10 +8,10 @@ object Macros {
     def defAndUseX(rhs: Tree) = {
       Block(List(ValDef(NoMods, newTermName("x"), TypeTree(), rhs)), x)
     }
-    val xi4 = defAndUseX(Literal(Constant(4)))
-    val xs2 = defAndUseX(Literal(Constant("2")))
-    c.Expr[String](Apply(Select(xi4, newTermName("$plus")), List(xs2)))
+    val xs2 = defAndUseX(Literal(Constant("4")))
+    val xi4 = defAndUseX(Literal(Constant(2)))
+    c.Expr[String](Apply(Select(xs2, newTermName("$plus")), List(xi4)))
   }
 
   def foo = macro impl
-}
+}

test/files/run/map_test.scala

@@ -30,7 +30,7 @@ object Test extends App {
     i = 0
     while(i < 4712) {
       if (map.isDefinedAt(i))
-        print(i + "->" + map(i) + " ");
+        print(s"$i->${map(i)} ")
       i += 1
     }
     println("")

test/files/run/names-defaults.scala

@@ -119,13 +119,13 @@ object Test extends App {
 
 
   // vararg
-  def test5(a: Int, b: Int)(c: Int, d: String*) = a +", "+ d.toList
+  def test5(a: Int, b: Int)(c: Int, d: String*) = s"$a, ${d.toList}"
   println(test5(b = 1, a = 2)(3, "4", "4", "4"))
   println(test5(b = 1, a = 2)(c = 29))
 
 
   // tuple conversion
-  def foo(a: Int, b: Int)(c: (Int, String)) = a + c._1
+  def foo(a: Int, b: Int)(c: (Int, String)) = s"$a${c._1}"
   println(foo(b = 1, a = 2)(3, "4"))
 
 
@@ -406,13 +406,13 @@ object Test extends App {
   println(f8177(a = 1, 1))
 
   // DEFINITIONS
-  def test1(a: Int, b: String) = println(a +": "+ b)
-  def test2(u: Int, v: Int)(k: String, l: Int) = println(l +": "+ k +", "+ (u + v))
+  def test1(a: Int, b: String) = println(s"$a: $b")
+  def test2(u: Int, v: Int)(k: String, l: Int) = println(l.toString +": "+ k +", "+ (u + v))
 
-  def test3[T1, T2](a: Int, b: T1)(c: String, d: T2) = println(a +": "+ c +", "+ b +", "+ d)
+  def test3[T1, T2](a: Int, b: T1)(c: String, d: T2) = println(a.toString +": "+ c +", "+ b +", "+ d)
 
   def test4(a: Int) = {
-    def inner(b: Int = a, c: String) = println(b +": "+ c)
+    def inner(b: Int = a, c: String) = println(b.toString +": "+ c)
     inner(c = "/")
   }
   def test5(argName: Unit) = println("test5")
@@ -425,13 +425,13 @@ object Test extends App {
 
 
 class Base {
-  def test1[T1, T2](a: Int = 100, b: T1)(c: T2, d: String = a +": "+ b)(e: T2 = c, f: Int) =
-    println(a +": "+ d +", "+ b +", "+ c +", "+ e +", "+ f)
+  def test1[T1, T2](a: Int = 100, b: T1)(c: T2, d: String = a.toString +": "+ b)(e: T2 = c, f: Int) =
+    println(a.toString +": "+ d +", "+ b +", "+ c +", "+ e +", "+ f)
 }
 
 class Sub1 extends Base {
   override def test1[U1, U2](b: Int, a: U1)(m: U2, r: String = "overridden")(o: U2, f: Int = 555) =
-    println(b +": "+ r +", "+ a +", "+ m +", "+ o +", "+ f)
+    println(b.toString +": "+ r +", "+ a +", "+ m +", "+ o +", "+ f)
 }