syntax: use Parser.matched consistently

It's the same behavior, but the code becomes more consistent. While here, make DebugPrint always finish the output with a newline.
mvdan · Jul 31, 2024 · e81efe5 · e81efe5
1 parent 3697a78
commit e81efe5
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/syntax/parser.go b/syntax/parser.go
@@ -1359,8 +1359,7 @@ func (p *Parser) paramExp() *ParamExp {
 		p.curErr("not a valid parameter expansion operator: %v", p.tok)
 	}
 	p.quote = old
-	pe.Rbrace = p.pos
-	p.matched(pe.Dollar, dollBrace, rightBrace)
+	pe.Rbrace = p.matched(pe.Dollar, dollBrace, rightBrace)
 	return pe
 }
 

diff --git a/syntax/walk.go b/syntax/walk.go
@@ -235,6 +235,7 @@ func Walk(node Node, f func(Node) bool) {
 func DebugPrint(w io.Writer, node Node) error {
 	p := debugPrinter{out: w}
 	p.print(reflect.ValueOf(node))
+	p.printf("\n")
 	return p.err
 }