diff options
Diffstat (limited to 'src/reflect')
-rw-r--r-- | src/reflect/scala/reflect/internal/Printers.scala | 35 |
1 files changed, 23 insertions, 12 deletions
diff --git a/src/reflect/scala/reflect/internal/Printers.scala b/src/reflect/scala/reflect/internal/Printers.scala index cb4abd8bd8..45be394231 100644 --- a/src/reflect/scala/reflect/internal/Printers.scala +++ b/src/reflect/scala/reflect/internal/Printers.scala @@ -73,11 +73,14 @@ trait Printers extends api.Printers { self: SymbolTable => def indent() = indentMargin += indentStep def undent() = indentMargin -= indentStep - def printPosition(tree: Tree) = if (printPositions) print(tree.pos.show) + def printPosition(tree: Tree) = + if (printPositions) comment(print(tree.pos.show)) protected def printTypesInfo(tree: Tree) = if (printTypes && tree.isTerm && tree.canHaveAttrs) - print("{", if (tree.tpe eq null) "<null>" else tree.tpe.toString, "}") + comment{ + print("{", if (tree.tpe eq null) "<null>" else tree.tpe.toString, "}") + } def println() = { out.println() @@ -128,12 +131,17 @@ trait Printers extends api.Printers { self: SymbolTable => print(symName(p, p.name)); printOpt(": ", TypeTree() setType p.tpe) } - protected def parenthesize(condition: Boolean = true)(body: => Unit) = { - if (condition) print("(") + protected def parenthesize(condition: Boolean = true, open: String = "(", close: String = ")")(body: => Unit) = { + if (condition) print(open) body - if (condition) print(")") + if (condition) print(close) } + protected val commentsRequired = false + + protected def comment(body: => Unit) = + parenthesize(commentsRequired, "/*", "*/")(body) + protected def printImplicitInParamsList(vds: List[ValDef]) = if (vds.nonEmpty) printFlags(vds.head.mods.flags & IMPLICIT, "") @@ -279,7 +287,11 @@ trait Printers extends api.Printers { self: SymbolTable => print("("); printValueParams print(" => ", body, ")") - if (printIds && tree.symbol != null) print("#" + tree.symbol.id) + if (printIds && tree.symbol != null) + comment{ + print("#" + tree.symbol.id) + } + if (printOwners && tree.symbol != null) print("@" + tree.symbol.owner.id) } @@ -547,6 +559,8 @@ trait Printers extends api.Printers { self: SymbolTable => qualIsIntLit && name.isOperatorName } + override protected val commentsRequired = true + protected def needsParentheses(parent: Tree)(insideIf: Boolean = true, insideMatch: Boolean = true, insideTry: Boolean = true, insideAnnotated: Boolean = true, insideBlock: Boolean = true, insideLabelDef: Boolean = true) = { parent match { @@ -1040,11 +1054,7 @@ trait Printers extends api.Printers { self: SymbolTable => case _ if syntheticToRemove(tree) => case tt: TypeTree => - if (printPositions) { - if (tt.original != null) - print("<type: ", tt.original.toString(), ">") - else print("<type ?>") - } else if (!emptyTree(tt)) print(tt.original) + if (!emptyTree(tt)) print(tt.original) // print only fun when targs are TypeTrees with empty original case TypeApply(fun, targs) => @@ -1076,7 +1086,7 @@ trait Printers extends api.Printers { self: SymbolTable => }) && (tr match { // check that Select contains package case Select(q, _) => checkRootPackage(q) case _: Ident | _: This => val sym = tr.symbol - tr.hasExistingSymbol && sym.isPackage && !sym.isRootPackage + tr.hasExistingSymbol && sym.isPackage && sym.name != nme.ROOTPKG case _ => false }) @@ -1093,6 +1103,7 @@ trait Printers extends api.Printers { self: SymbolTable => } case This(qual) => + // todo: add symbol checking for common printer if (tree.symbol.isPackage) print(tree.symbol.fullName) else super.processTreePrinting(tree) |