diff options
author | Paul Phillips <paulp@improving.org> | 2011-07-12 19:42:50 +0000 |
---|---|---|
committer | Paul Phillips <paulp@improving.org> | 2011-07-12 19:42:50 +0000 |
commit | 6163cdcc236698b333016becc57f545098760e32 (patch) | |
tree | 0c0358a47d20ae87418982e8cdae0ced4b16539d | |
parent | 5eefefb73b8ab01a442ddf946e86df410b528771 (diff) | |
download | scala-6163cdcc236698b333016becc57f545098760e32.tar.gz scala-6163cdcc236698b333016becc57f545098760e32.tar.bz2 scala-6163cdcc236698b333016becc57f545098760e32.zip |
Fixed some unnecessary chattiness and poor func...
Fixed some unnecessary chattiness and poor function naming in tree
checkers, no review.
-rw-r--r-- | src/compiler/scala/tools/nsc/typechecker/TreeCheckers.scala | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/src/compiler/scala/tools/nsc/typechecker/TreeCheckers.scala b/src/compiler/scala/tools/nsc/typechecker/TreeCheckers.scala index 75483f777c..a8c3a13646 100644 --- a/src/compiler/scala/tools/nsc/typechecker/TreeCheckers.scala +++ b/src/compiler/scala/tools/nsc/typechecker/TreeCheckers.scala @@ -68,12 +68,12 @@ abstract class TreeCheckers extends Analyzer { def reportChanges(): Unit = { // new symbols if (newSyms.nonEmpty) { - val str = - if (settings.debug.value) "New symbols: " + (sortedNewSyms mkString " ") - else newSyms.size + " new symbols." + informFn(newSyms.size + " new symbols.") + val toPrint = if (settings.debug.value) sortedNewSyms mkString " " else "" newSyms.clear() - errorFn(str) + if (toPrint != "") + informFn(toPrint) } // moved symbols @@ -118,6 +118,10 @@ abstract class TreeCheckers extends Analyzer { def errorFn(msg: Any): Unit = println("[check: %s] %s".format(phase.prev, msg)) def errorFn(pos: Position, msg: Any): Unit = errorFn(posstr(pos) + ": " + msg) + def informFn(msg: Any) { + if (settings.verbose.value || settings.debug.value) + println("[check: %s] %s".format(phase.prev, msg)) + } def assertFn(cond: Boolean, msg: => Any) = if (!cond) errorFn(msg) |