Permalink
Browse files

Don't trace the low-level details of ResetAttrs under -Ydebug

This need for this output is highly situational and since we've
starting using this resettter in case classes and names/defaults,
it is polluting our consoles.
  • Loading branch information...
1 parent b7b210d commit c258ccc9b52c9bfcd47342d3dcf6b58faac8535b @retronym retronym committed Jan 17, 2014
Showing with 3 additions and 1 deletion.
  1. +3 −1 src/compiler/scala/tools/nsc/ast/Trees.scala
@@ -194,7 +194,9 @@ trait Trees extends scala.reflect.internal.Trees { self: Global =>
* (bq:) This transformer has mutable state and should be discarded after use
*/
private class ResetAttrs(localOnly: Boolean, leaveAlone: Tree => Boolean = null, keepLabels: Boolean = false) {
- val debug = settings.debug.value
+ // this used to be based on -Ydebug, but the need for logging in this code is so situational
+ // that I've reverted to a hard-coded constant here.
+ val debug = false
val trace = scala.tools.nsc.util.trace when debug
val locals = util.HashSet[Symbol](8)

0 comments on commit c258ccc

Please sign in to comment.