SI-8113 allow a newline between a link target and title

Parsing "[[http://foo.bar link title]]" stops at the first whitespace.
This breaks pretty badly in:

[[http://foo.bar
link title]]

It stops after "link", interprets what it parsed as a link to a member,
obviously fails, and then just ouputs "title".
It should at least return a proper error, or, even better, just allow a
newline between the target and title. I went for the latter.

2 files changed, 37 insertions, 0 deletions

diff --git a/test/scaladoc/run/t8113.check b/test/scaladoc/run/t8113.check
new file mode 100644
index 0000000000..619c56180b
--- /dev/null
+++ b/test/scaladoc/run/t8113.check
@@ -0,0 +1 @@
+Done.
diff --git a/test/scaladoc/run/t8113.scala b/test/scaladoc/run/t8113.scala
new file mode 100644
index 0000000000..f006213ef2
--- /dev/null
+++ b/test/scaladoc/run/t8113.scala
@@ -0,0 +1,36 @@
+import scala.tools.nsc.doc.base._
+import scala.tools.nsc.doc.base.comment._
+import scala.tools.nsc.doc.model._
+import scala.tools.partest.ScaladocModelTest
+
+object Test extends ScaladocModelTest {
+
+  override def code = """
+  /**
+   *  Check out [[http://www.scala-lang.org
+   *  this great website]]!
+   */
+  class Test
+  """
+
+  def scaladocSettings = ""
+
+  def testModel(rootPackage: Package) = {
+    import access._
+
+    val test = rootPackage._class("Test")
+
+    // find Link
+    def find(body: Any): Option[Link] = body match {
+        case l: Link => Some(l)
+        case s: Seq[_]  => s.toList.map(find(_)).flatten.headOption
+        case p: Product => p.productIterator.toList.map(find(_)).flatten.headOption
+        case _          => None
+      }
+
+    val link = find(test.comment.get.body).collect { case Link(ta, Text(ti)) => (ta, ti) }
+    assert(link.isDefined)
+    val expected = ("http://www.scala-lang.org", "this great website")
+    link.foreach {l => assert(l == expected, s"$l != $expected")}
+  }
+}