From d367c5f925e7055e9ee5b4071328e92cf48ce0ed Mon Sep 17 00:00:00 2001
From: Tim Harper <timcharper@gmail.com>
Date: Wed, 29 Oct 2014 00:15:43 -0600
Subject: Fixes memory leak when using reflection

References to Threads would be retained long after their termination if
reflection is used in them. This led to a steady, long memory leak in
applications using reflection in thread pools.
---
 test/files/run/t8946.scala | 29 +++++++++++++++++++++++++++++
 1 file changed, 29 insertions(+)
 create mode 100644 test/files/run/t8946.scala

(limited to 'test/files')

diff --git a/test/files/run/t8946.scala b/test/files/run/t8946.scala
new file mode 100644
index 0000000000..a248a20501
--- /dev/null
+++ b/test/files/run/t8946.scala
@@ -0,0 +1,29 @@
+// Tests to assert that references to threads are not strongly held when scala-reflection is used inside of them.
+object Test {
+  import scala.ref.WeakReference
+
+  def forceGc() = {
+    var obj = new Object
+    val ref = new WeakReference(obj)
+    obj = null;
+    while(ref.get.nonEmpty)
+      Array.ofDim[Byte](16 * 1024 * 1024)
+  }
+
+  def main(args: Array[String]): Unit = {
+    val threads = for (i <- (1 to 16)) yield {
+      val t = new Thread {
+        override def run(): Unit = {
+          import reflect.runtime.universe._
+          typeOf[List[String]] <:< typeOf[Seq[_]]
+        }
+      }
+      t.start()
+      t.join()
+      WeakReference(t)
+    }
+    forceGc()
+    val nonGCdThreads = threads.filter(_.get.nonEmpty).length
+    assert(nonGCdThreads == 0, s"${nonGCdThreads} threads were retained; expected 0.")
+  }
+}
-- 
cgit v1.2.3