Add location information to group AST node (#112)

Mingun · web-flow · commit 55e04a732774 · 2021-04-23T15:32:02.000-06:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -15,6 +15,11 @@ Released: TBD
   data,for example, path to the currently parsed file.
   [@Mingun](https://github.com/peggyjs/peggy/pull/95)
 
+### Bug fixes
+
+- [#112](https://github.com/peggyjs/peggy/pull/112): `"group"` node in the AST now have `location` information (back-ported)
+
+
 1.1.0
 -----
 
diff --git a/lib/parser.js b/lib/parser.js
diff --git a/src/parser.pegjs b/src/parser.pegjs
@@ -201,7 +201,7 @@ PrimaryExpression
       // nodes that already isolate label scope themselves. This leaves us with
       // "labeled" and "sequence".
       return expression.type === "labeled" || expression.type === "sequence"
-          ? { type: "group", expression: expression }
+          ? { type: "group", expression: expression, location: location() }
           : expression;
     }
 

Original file line number	Diff line number	Diff line change
`@@ -201,7 +201,7 @@ PrimaryExpression`
`201`	`201`	`// nodes that already isolate label scope themselves. This leaves us with`
`202`	`202`	`// "labeled" and "sequence".`
`203`	`203`	`return expression.type === "labeled" \|\| expression.type === "sequence"`
`204`		`- ? { type: "group", expression: expression }`
	`204`	`+ ? { type: "group", expression: expression, location: location() }`
`205`	`205`	`: expression;`
`206`	`206`	`}`
`207`	`207`