Return Syntax, rather than SyntaxProtocol, from foldAll

DougGregor · DougGregor · commit f7d924a6bd5e · 2022-09-06T23:33:13.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
@@ -75,7 +75,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
`@@ -75,7 +75,7 @@ func foldAllSequences(_ tree: SourceFileSyntax) -> (Syntax, [Diagnostic]) {`
`75`	`75`	`var operatorTable = OperatorTable.standardOperators`
`76`	`76`	`operatorTable.addSourceFile(tree, errorHandler: recordOperatorError)`
`77`	`77`	`let resultTree = operatorTable.foldAll(tree, errorHandler: recordOperatorError)`
`78`		`- return (Syntax(resultTree), diags)`
	`78`	`+ return (resultTree, diags)`
`79`	`79`	`}`
`80`	`80`
`81`	`81`	`class VerifyRoundTrip: ParsableCommand {`