From 53cd512eef8f9f28527e7c72d108359f0313d3f5 Mon Sep 17 00:00:00 2001 From: Martin Odersky Date: Fri, 19 Dec 2014 14:33:35 +0100 Subject: [PATCH] Disable context escape detection I get very frequent build failures due to the mutual Scala<->Java dependency. It builds for a long time, then decides that getCtx overrides nothing. The only way to fix is another clean build. Total time lost: >5 minutes. These happened occasionally before but have become much more frequent under ScalaIDe4.0, to the point where this becomes a major drag on productity. Context escape detection is nice but if it stops us getting work done, not worth the effort. --- test/test/DottyTest.scala | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/test/test/DottyTest.scala b/test/test/DottyTest.scala index 90a0154ec1cd..21eebd660c59 100644 --- a/test/test/DottyTest.scala +++ b/test/test/DottyTest.scala @@ -14,7 +14,7 @@ import dotty.tools.dotc.Compiler import dotty.tools.dotc import dotty.tools.dotc.core.Phases.Phase -class DottyTest extends ContextEscapeDetection{ +class DottyTest /*extends ContextEscapeDetection*/ { dotty.tools.dotc.parsing.Scanners // initialize keywords @@ -36,11 +36,12 @@ class DottyTest extends ContextEscapeDetection{ base.definitions.init(ctx) ctx } - +/* override def getCtx: Context = ctx override def clearCtx() = { ctx = null } +*/ private def compilerWithChecker(phase: String)(assertion:(tpd.Tree, Context) => Unit) = new Compiler { self => override def phases = {