SI-8210 Scaladoc: Fix the false negative @inheritdoc warning on acces…

…sors

This fix is just for the false negative warning. Probably we can skip
setters entirely, but I'm not 100% sure.

src/compiler/scala/tools/nsc/ast/DocComments.scala

@@ -76,9 +76,9 @@ trait DocComments { self: Global =>
 
     superComment(sym) match {
       case None =>
-        if (ownComment.indexOf("@inheritdoc") != -1)
-          reporter.warning(sym.pos, "The comment for " + sym +
-              " contains @inheritdoc, but no parent comment is available to inherit from.")
+        // SI-8210 - The warning would be false negative when this symbol is a setter
+        if (ownComment.indexOf("@inheritdoc") != -1 && ! sym.isSetter)
+          reporter.warning(sym.pos, s"The comment for ${sym} contains @inheritdoc, but no parent comment is available to inherit from.")
         ownComment.replaceAllLiterally("@inheritdoc", "<invalid inheritdoc annotation>")
       case Some(sc) =>
         if (ownComment == "") sc

test/scaladoc/run/SI-8210.check

@@ -0,0 +1 @@
+Done.

test/scaladoc/run/SI-8210.scala

@@ -0,0 +1,24 @@
+import scala.tools.nsc.doc.model._
+import scala.tools.partest.ScaladocModelTest
+
+object Test extends ScaladocModelTest {
+  override def code = """
+object Foo {
+  trait Config {
+    /** The bar obviously. */
+    def bar: Int
+  }
+  class ConfigBuilder extends Config {
+    /** @inheritdoc
+      *
+      * The default value is 1234.
+      */
+    var bar: Int = 1234
+  }
+}
+  """
+
+  def scaladocSettings = ""
+
+  def testModel(root: Package) = ()
+}