SI-7814 Avoid init cycle between Predef, `package`, ScalaRuntime

Not every application will force these in a single thread; we
have to do our best to avoid cycles between them.

The enclosed test was failing every time before the change.

This commit breaks the cycle by avoiding computing `tupleNames`
in the constructor of `ScalaRuntime`. The new version has the
added benefit of including specialized tuple subclasses, which
is verified with a unit test for `isTuple`.

Are there more of these lurking? It seems likely. I'm more than
a little concerned about the way the `ControlThrowable` fires up
`scala.SystemProperties` to check whether or not to suppress
stack traces; there is already an ugly hack in place:

    object NoStackTrace {
      final def noSuppression = _noSuppression

      // two-stage init to make checkinit happy,
      // since sys.SystemProperties.noTraceSupression.value
      // calls back into NoStackTrace.noSuppression
      final private var _noSuppression = false
      _noSuppression = sys.SystemProperties.noTraceSupression.value
    }

1 files changed, 70 insertions, 0 deletions

diff --git a/test/junit/scala/runtime/ScalaRunTimeTest.scala b/test/junit/scala/runtime/ScalaRunTimeTest.scala
new file mode 100644
index 0000000000..9da197c71a
--- /dev/null
+++ b/test/junit/scala/runtime/ScalaRunTimeTest.scala
@@ -0,0 +1,70 @@
+package scala.runtime
+
+import org.junit.Assert._
+import org.junit.Test
+import org.junit.runner.RunWith
+import org.junit.runners.JUnit4
+
+/** Tests for the private class DefaultPromise */
+@RunWith(classOf[JUnit4])
+class ScalaRunTimeTest {
+  @Test
+  def testIsTuple() {
+    import ScalaRunTime.isTuple
+    def check(v: Any) = {
+      assertTrue(v.toString, isTuple(v))
+    }
+
+    val s = ""
+    check(Tuple1(s))
+    check((s, s))
+    check((s, s, s))
+    check((s, s, s, s))
+    check((s, s, s, s, s))
+    check((s, s, s, s, s, s))
+    check((s, s, s, s, s, s, s))
+    check((s, s, s, s, s, s, s, s))
+    check((s, s, s, s, s, s, s, s, s))
+    check((s, s, s, s, s, s, s, s, s, s))
+    check((s, s, s, s, s, s, s, s, s, s, s))
+    check((s, s, s, s, s, s, s, s, s, s, s, s))
+    check((s, s, s, s, s, s, s, s, s, s, s, s, s))
+    check((s, s, s, s, s, s, s, s, s, s, s, s, s, s))
+    check((s, s, s, s, s, s, s, s, s, s, s, s, s, s, s))
+    check((s, s, s, s, s, s, s, s, s, s, s, s, s, s, s, s))
+    check((s, s, s, s, s, s, s, s, s, s, s, s, s, s, s, s, s))
+    check((s, s, s, s, s, s, s, s, s, s, s, s, s, s, s, s, s, s))
+    check((s, s, s, s, s, s, s, s, s, s, s, s, s, s, s, s, s, s, s))
+    check((s, s, s, s, s, s, s, s, s, s, s, s, s, s, s, s, s, s, s, s))
+    check((s, s, s, s, s, s, s, s, s, s, s, s, s, s, s, s, s, s, s, s, s))
+    check((s, s, s, s, s, s, s, s, s, s, s, s, s, s, s, s, s, s, s, s, s, s))
+
+    // some specialized variants will have mangled classnames
+    check(Tuple1(0))
+    check((0, 0))
+    check((0, 0, 0))
+    check((0, 0, 0, 0))
+    check((0, 0, 0, 0, 0))
+    check((0, 0, 0, 0, 0, 0))
+    check((0, 0, 0, 0, 0, 0, 0))
+    check((0, 0, 0, 0, 0, 0, 0, 0))
+    check((0, 0, 0, 0, 0, 0, 0, 0, 0))
+    check((0, 0, 0, 0, 0, 0, 0, 0, 0, 0))
+    check((0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0))
+    check((0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0))
+    check((0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0))
+    check((0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0))
+    check((0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0))
+    check((0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0))
+    check((0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0))
+    check((0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0))
+    check((0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0))
+    check((0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0))
+    check((0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0))
+    check((0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0))
+
+    case class C()
+    val c = new C()
+    assertFalse(c.toString, isTuple(c))
+  }
+}