Return Syntax, rather than SyntaxProtocol, from foldAll

DougGregor · DougGregor · commit 1c17c419ffb6 · 2022-09-16T15:32:58.000-07:00
diff --git a/Sources/SwiftOperators/OperatorTable+Folding.swift b/Sources/SwiftOperators/OperatorTable+Folding.swift
@@ -453,7 +453,7 @@ extension OperatorTable {
   public func foldAll<Node: SyntaxProtocol>(
     _ node: Node,
     errorHandler: OperatorErrorHandler = { throw $0 }
-  ) rethrows -> SyntaxProtocol {
+  ) rethrows -> Syntax {
     return try withoutActuallyEscaping(errorHandler) { errorHandler in
       let folder = SequenceFolder(
         opPrecedence: self, errorHandler: errorHandler
diff --git a/Sources/swift-parser-test/swift-parser-test.swift b/Sources/swift-parser-test/swift-parser-test.swift
@@ -67,7 +67,7 @@ func foldAllSequences(_ tree: SourceFileSyntax) -> (Syntax, [Diagnostic]) {
   var operatorTable = OperatorTable.standardOperators
   operatorTable.addSourceFile(tree, errorHandler: recordOperatorError)
   let resultTree = operatorTable.foldAll(tree, errorHandler: recordOperatorError)
-  return (Syntax(resultTree), diags)
+  return (resultTree, diags)
 }
 
 class VerifyRoundTrip: ParsableCommand {

Original file line number	Diff line number	Diff line change
`@@ -67,7 +67,7 @@ func foldAllSequences(_ tree: SourceFileSyntax) -> (Syntax, [Diagnostic]) {`
`67`	`67`	`var operatorTable = OperatorTable.standardOperators`
`68`	`68`	`operatorTable.addSourceFile(tree, errorHandler: recordOperatorError)`
`69`	`69`	`let resultTree = operatorTable.foldAll(tree, errorHandler: recordOperatorError)`
`70`		`- return (Syntax(resultTree), diags)`
	`70`	`+ return (resultTree, diags)`
`71`	`71`	`}`
`72`	`72`
`73`	`73`	`class VerifyRoundTrip: ParsableCommand {`