Position#show prints the point.

Can't ensure range position points are meaningful when we never
see them. To limit noise, only print the point when it != start.

  [x:y]    // point=x, start=x, end=y
  [p/x:y]  // point=p, start=x, end=y

I'm open to a different syntax.

Also prints NoPosition as [X] rather than [NoPosition] because
noise is for construction workers and attenders of rock concerts.

Some range position and parser tests are included so we can see
the checkfile change when the forthcoming fix happens (either an
error message improvement or a positional one.)

1 files changed, 3 insertions, 3 deletions

diff --git a/test/files/run/t6288.check b/test/files/run/t6288.check
index a032a10de6..0da03c0439 100644
--- a/test/files/run/t6288.check
+++ b/test/files/run/t6288.check
@@ -18,7 +18,7 @@
       [64]case6(){
         [64][64]matchEnd4([64]throw [64][64][64]new [64]MatchError([64]x1))
       };
-      [64]matchEnd4(x: [NoPosition]Unit){
+      [64]matchEnd4(x: [X]Unit){
         [64]x
       }
     }
@@ -44,7 +44,7 @@
       [175]case6(){
         [175][175]matchEnd4([175]throw [175][175][175]new [175]MatchError([175]x1))
       };
-      [175]matchEnd4(x: [NoPosition]Unit){
+      [175]matchEnd4(x: [X]Unit){
         [175]x
       }
     }
@@ -70,7 +70,7 @@
       [273]case6(){
         [273][273]matchEnd4([273]throw [273][273][273]new [273]MatchError([273]x1))
       };
-      [273]matchEnd4(x: [NoPosition]Unit){
+      [273]matchEnd4(x: [X]Unit){
         [273]x
       }
     }