Merge pull request #2083 from dotty-staging/fix/plainprinter-pos

smarter · web-flow · commit e5e2f826b2db · 2017-03-12T20:16:44.000+01:00
-Yplain-printer: do not print positions by default
diff --git a/compiler/src/dotty/tools/dotc/printing/PlainPrinter.scala b/compiler/src/dotty/tools/dotc/printing/PlainPrinter.scala
@@ -483,7 +483,7 @@ class PlainPrinter(_ctx: Context) extends Printer {
           else
             Text()
 
-        nodeName ~ "(" ~ elems ~ tpSuffix ~ ")" ~ node.pos.toString
+        nodeName ~ "(" ~ elems ~ tpSuffix ~ ")" ~ (node.pos.toString provided ctx.settings.Yprintpos.value)
       case _ =>
         tree.fallbackToText(this)
     }

Original file line number	Diff line number	Diff line change
`@@ -483,7 +483,7 @@ class PlainPrinter(_ctx: Context) extends Printer {`
`483`	`483`	`else`
`484`	`484`	`Text()`
`485`	`485`
`486`		`- nodeName ~ "(" ~ elems ~ tpSuffix ~ ")" ~ node.pos.toString`
	`486`	`+ nodeName ~ "(" ~ elems ~ tpSuffix ~ ")" ~ (node.pos.toString provided ctx.settings.Yprintpos.value)`
`487`	`487`	`case _ =>`
`488`	`488`	`tree.fallbackToText(this)`
`489`	`489`	`}`