Don't populate the concreteSpecMethod set in presentation compiler runs. This is safe because the presentation

dragos · dragos · commit 92291e72b165 · 2012-02-22T20:53:17.000+01:00
compiler never runs the tree transformer (where this map is needed). This is a source of serious memory leaks in the IDE, but it wasn't visible before because the IDE didn't run the info transformers far enough. Interestingly, this is not a leak in batch runs: each element of the set is removed when the tree is transformed. For a nice graph of the effect of this change, see: http://i41.tinypic.com/xe0k7o.jpg
diff --git a/src/compiler/scala/tools/nsc/transform/SpecializeTypes.scala b/src/compiler/scala/tools/nsc/transform/SpecializeTypes.scala
@@ -1719,7 +1719,7 @@ abstract class SpecializeTypes extends InfoTransform with TypingTransformers {
    *        that here are not garbage collected at the end of a compiler run!
    */
   def addConcreteSpecMethod(m: Symbol) {
-    if (currentRun.compiles(m)) concreteSpecMethods += m
+    if (!forInteractive && currentRun.compiles(m)) concreteSpecMethods += m
   }
 
   private def makeArguments(fun: Symbol, vparams: List[Symbol]): List[Tree] = (

Original file line number	Diff line number	Diff line change
`@@ -1719,7 +1719,7 @@ abstract class SpecializeTypes extends InfoTransform with TypingTransformers {`
`1719`	`1719`	`* that here are not garbage collected at the end of a compiler run!`
`1720`	`1720`	`*/`
`1721`	`1721`	`def addConcreteSpecMethod(m: Symbol) {`
`1722`		`- if (currentRun.compiles(m)) concreteSpecMethods += m`
	`1722`	`+ if (!forInteractive && currentRun.compiles(m)) concreteSpecMethods += m`
`1723`	`1723`	`}`
`1724`	`1724`
`1725`	`1725`	`private def makeArguments(fun: Symbol, vparams: List[Symbol]): List[Tree] = (`