From b3983a403070aa1b693557a931dbd08bf2bfa56e Mon Sep 17 00:00:00 2001 From: Dmitry Petrashko Date: Wed, 2 Apr 2014 18:08:52 +0200 Subject: [PATCH] Reduce verbosity of logs. We are already over limit on output size imposed by travis that is shown in webpage, if we'll continue to add tests will be soon over limit even to run builds. This commit disables printing of classpath, and removes printlns in several places. In order for ShowClassTests to print info as is was printing previously, please set "test.ShowClassTests.verbose" property. --- src/dotty/tools/dotc/transform/Splitter.scala | 2 +- test/test/DottyTest.scala | 5 ++--- test/test/ShowClassTests.scala | 2 +- test/test/transform/CreateCompanionObjectsTest.scala | 1 - 4 files changed, 4 insertions(+), 6 deletions(-) diff --git a/src/dotty/tools/dotc/transform/Splitter.scala b/src/dotty/tools/dotc/transform/Splitter.scala index 9c01574aae45..43a91a058c51 100644 --- a/src/dotty/tools/dotc/transform/Splitter.scala +++ b/src/dotty/tools/dotc/transform/Splitter.scala @@ -20,7 +20,7 @@ class Splitter extends TreeTransform { /** Replace self referencing idents with ThisTypes. */ override def transformIdent(tree: Ident)(implicit ctx: Context, info: TransformerInfo) = tree.tpe match { case ThisType(cls) => - println(s"owner = ${ctx.owner}, context = ${ctx}") + ctx.debuglog(s"owner = ${ctx.owner}, context = ${ctx}") This(cls) withPos tree.pos case _ => tree } diff --git a/test/test/DottyTest.scala b/test/test/DottyTest.scala index 604f5d500b7f..3c308e230b5c 100644 --- a/test/test/DottyTest.scala +++ b/test/test/DottyTest.scala @@ -22,18 +22,17 @@ class DottyTest { val base = new ContextBase import base.settings._ val ctx = base.initialCtx.fresh - .setSetting(verbose, true) + //.setSetting(verbose, true) // .withSetting(debug, true) // .withSetting(debugTrace, true) // .withSetting(prompt, true) - .setSetting(Ylogcp, true) + //.setSetting(Ylogcp, true) .setSetting(printtypes, true) .setSetting(pageWidth, 90) .setSetting(log, List(" Any) = { - if (!sys.props.isDefinedAt("dotty.travis.build")) + if (sys.props.isDefinedAt("test.ShowClassTests.verbose")) println(msg) } diff --git a/test/test/transform/CreateCompanionObjectsTest.scala b/test/test/transform/CreateCompanionObjectsTest.scala index bee9b414e004..0a50cb9420c8 100644 --- a/test/test/transform/CreateCompanionObjectsTest.scala +++ b/test/test/transform/CreateCompanionObjectsTest.scala @@ -37,7 +37,6 @@ class CreateCompanionObjectsTest extends DottyTest { override def name: String = "test" } val transformed = transformer.transform(tree).toString - println(transformed) val classPattern = "TypeDef(Modifiers(,,List()),A," val classPos = transformed.indexOf(classPattern) val moduleClassPattern = "TypeDef(Modifiers(final module ,,List()),A$"