Respect -deprecation flag

Author: liufengyun

compiler/src/dotty/tools/dotc/reporting/Reporter.scala

@@ -50,7 +50,7 @@ trait Reporting { this: Context =>
     }
 
   def deprecationWarning(msg: => Message, pos: SourcePosition = NoSourcePosition): Unit =
-    reportWarning(new DeprecationWarning(msg, pos))
+    if (this.settings.deprecation.value) reportWarning(new DeprecationWarning(msg, pos))
 
   def migrationWarning(msg: => Message, pos: SourcePosition = NoSourcePosition): Unit =
     reportWarning(new MigrationWarning(msg, pos))

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

@@ -146,6 +146,7 @@ class CompilationTests extends ParallelTesting {
     compileFilesInDir("tests/neg", defaultOptions) +
     compileFilesInDir("tests/neg-tailcall", defaultOptions) +
     compileFilesInDir("tests/neg-kind-polymorphism", defaultOptions and "-Ykind-polymorphism") +
+    compileDir("tests/neg-custom-args/deprecate", defaultOptions.and("-Xfatal-warnings", "-deprecation")) +
     compileFilesInDir("tests/neg-custom-args/fatal-warnings", defaultOptions.and("-Xfatal-warnings")) +
     compileFilesInDir("tests/neg-custom-args/allow-double-bindings", allowDoubleBindings) +
     compileDir("tests/neg-custom-args/impl-conv", defaultOptions.and("-Xfatal-warnings", "-feature")) +

tests/neg-custom-args/deprecation/A_1.scala

@@ -0,0 +1,5 @@
+object A {
+  @deprecated("use bar instead of this one", "0.2.3")
+  def foo: String = ""
+  def bar: String = ""
+}

tests/neg-custom-args/deprecation/B_2.scala

@@ -0,0 +1,3 @@
+class B {
+  A.foo // error: deprecation
+}

tests/pos-special/fatal-warnings/deprecation.scala

@@ -0,0 +1,8 @@
+object A {
+  @deprecated("use bar instead of this one", "0.2.3")
+  def foo: Int = 3
+}
+
+object B {
+  A.foo
+}