Revert/fix warnings changes on pending/run tests

Author: OlivierBlanvillain

compiler/test/dotty/tools/dotc/CompilationTests.scala

@@ -194,7 +194,8 @@ class CompilationTests extends ParallelTesting {
     compileFile("tests/neg-custom-args/phantom-overload.scala", allowDoubleBindings) +
     compileFile("tests/neg-custom-args/phantom-overload-2.scala", allowDoubleBindings) +
     compileFile("tests/neg-custom-args/structural.scala", defaultOptions.and("-Xfatal-warnings")) +
-    compileFile("tests/neg-custom-args/i2673.scala", defaultOptions.and("-Xfatal-warnings"))
+    compileFile("tests/neg-custom-args/i2673.scala", defaultOptions.and("-Xfatal-warnings")) +
+    compileFile("tests/neg-custom-args/t3235-minimal.scala", defaultOptions.and("-Xfatal-warnings"))
   }.checkExpectedErrors()
 
   // Run tests -----------------------------------------------------------------

tests/neg-custom-args/t3235-minimal.scala

@@ -0,0 +1,8 @@
+object Test {
+  def main(args: Array[String]): Unit = {
+    assert(123456789.round == 123456789) // error
+    assert(math.round(123456789) == 123456789) // error
+    assert(1234567890123456789L.round == 1234567890123456789L) // error
+    assert(math.round(1234567890123456789L) == 1234567890123456789L) // error
+  }
+}

tests/pending/run/t1368.check

@@ -0,0 +1,3 @@
+t1368.scala:7: warning: Reference to uninitialized value blurp
+  def go3 = (new AnyRef with Happy with Sad { override val status = blurp ; val blurp = "happysad" }).status
+                                                                    ^

tests/run/t1368.scala renamed to tests/pending/run/t1368.scala

@@ -0,0 +1,6 @@
+t2106.scala:7: warning: Could not inline required method foo because access level required by callee not matched by caller.
+  def main(args: Array[String]): Unit = x.foo
+                                          ^
+t2106.scala:7: warning: At the end of the day, could not inline @inline-marked method foo
+  def main(args: Array[String]): Unit = x.foo
+                                          ^

tests/pending/run/t2106.check

@@ -0,0 +1,17 @@
+t4047.scala:23: warning: a pure expression does nothing in statement position; you may be omitting necessary parentheses
+  a.foo
+    ^
+t4047.scala:24: warning: a pure expression does nothing in statement position; you may be omitting necessary parentheses
+  a.foo
+    ^
+t4047.scala:26: warning: a pure expression does nothing in statement position; you may be omitting necessary parentheses
+  b.foo
+    ^
+t4047.scala:27: warning: a pure expression does nothing in statement position; you may be omitting necessary parentheses
+  b.foo
+    ^
+Unit: called A.foo
+Unit: called B.foo
+Unit: called C.foo
+Unit: called D.foo
+Unit: called D.foo

tests/run/t2106.scala renamed to tests/pending/run/t2106.scala

@@ -1,4 +1,4 @@
-trait Foo[T] { def foo: T}
+trait Foo[T] { val foo: T}
 
 class A extends Foo[Unit]{
   lazy val foo = println("Unit: called A.foo")

tests/pending/run/t4047.check

@@ -0,0 +1,9 @@
+t5380.scala:3: warning: a pure expression does nothing in statement position; you may be omitting necessary parentheses
+    val f = () => return try { 1 } catch { case _: Throwable => 0 }
+                               ^
+t5380.scala:3: warning: a pure expression does nothing in statement position; you may be omitting necessary parentheses
+    val f = () => return try { 1 } catch { case _: Throwable => 0 }
+                                                                ^
+t5380.scala:3: warning: enclosing method main has result type Unit: return value discarded
+    val f = () => return try { 1 } catch { case _: Throwable => 0 }
+                  ^

tests/run/t4047.scala renamed to tests/pending/run/t4047.scala

@@ -0,0 +1,6 @@
+t7290.scala:4: warning: Pattern contains duplicate alternatives: 0
+    case 0 | 0 => 0
+         ^
+t7290.scala:5: warning: Pattern contains duplicate alternatives: 2, 3
+    case 2 | 2 | 2 | 3 | 2 | 3 => 0
+         ^

tests/pending/run/t5380.check

@@ -1,3 +1,24 @@
+t8266-octal-interp.scala:4: warning: Octal escape literals are deprecated, use \b instead.
+    f"a\10c",
+       ^
+t8266-octal-interp.scala:5: warning: Octal escape literals are deprecated, use \t instead.
+    f"a\11c",
+       ^
+t8266-octal-interp.scala:6: warning: Octal escape literals are deprecated, use \n instead.
+    f"a\12c",
+       ^
+t8266-octal-interp.scala:7: warning: Octal escape literals are deprecated, use \r instead.
+    f"a\15c",
+       ^
+t8266-octal-interp.scala:8: warning: Octal escape literals are deprecated, use ${'"'} or a triple-quoted literal """with embedded " or \u0022""" instead.
+    f"a\42c",
+       ^
+t8266-octal-interp.scala:9: warning: Octal escape literals are deprecated, use \\ instead.
+    f"a\134c",
+       ^
+t8266-octal-interp.scala:10: warning: Octal escape literals are deprecated, use \u0069 instead.
+    f"a\15151515c"
+       ^
 ac
 a	c
 a