Fix the .pos.point of Bind nodes

smarter · smarter · commit 1841b3db0e5d · 2017-12-06T16:55:28.000+01:00
The point of a definition should be the position of the first letter in
the name of a definition. For a `Bind` node this is just the
start (previously we were using the position of the `:`).
diff --git a/compiler/src/dotty/tools/dotc/parsing/Parsers.scala b/compiler/src/dotty/tools/dotc/parsing/Parsers.scala
@@ -1181,7 +1181,8 @@ object Parsers {
         t
     }
 
-    def ascription(t: Tree, location: Location.Value) = atPos(startOffset(t), in.skipToken()) {
+    def ascription(t: Tree, location: Location.Value) = atPos(startOffset(t)) {
+      in.skipToken()
       in.token match {
         case USCORE =>
           val uscoreStart = in.skipToken()

Original file line number	Diff line number	Diff line change
`@@ -1181,7 +1181,8 @@ object Parsers {`
`1181`	`1181`	`t`
`1182`	`1182`	`}`
`1183`	`1183`
`1184`		`- def ascription(t: Tree, location: Location.Value) = atPos(startOffset(t), in.skipToken()) {`
	`1184`	`+ def ascription(t: Tree, location: Location.Value) = atPos(startOffset(t)) {`
	`1185`	`+ in.skipToken()`
`1185`	`1186`	`in.token match {`
`1186`	`1187`	`case USCORE =>`
`1187`	`1188`	`val uscoreStart = in.skipToken()`