Fix spacings around !and ?

CodeGeneration/Sources/SyntaxSupport/gyb_generated/TokenSpec.swift

@@ -190,8 +190,8 @@ public let SYNTAX_TOKENS: [TokenSpec] = [
   PunctuatorSpec(name: "Arrow", kind: "arrow", text: "->", requiresLeadingSpace: true, requiresTrailingSpace: true),
   PunctuatorSpec(name: "Backtick", kind: "backtick", text: "`"),
   PunctuatorSpec(name: "Backslash", kind: "backslash", text: "\\"),
-  PunctuatorSpec(name: "ExclamationMark", kind: "exclaim_postfix", text: "!", requiresTrailingSpace: true),
-  PunctuatorSpec(name: "PostfixQuestionMark", kind: "question_postfix", text: "?", requiresTrailingSpace: true),
+  PunctuatorSpec(name: "ExclamationMark", kind: "exclaim_postfix", text: "!"),
+  PunctuatorSpec(name: "PostfixQuestionMark", kind: "question_postfix", text: "?"),
   PunctuatorSpec(name: "InfixQuestionMark", kind: "question_infix", text: "?"),
   PunctuatorSpec(name: "StringQuote", kind: "string_quote", text: "\"", classification: "StringLiteral"),
   PunctuatorSpec(name: "SingleQuote", kind: "single_quote", text: "\'", classification: "StringLiteral"),

CodeGeneration/Sources/generate-swiftsyntax/templates/basicformat/BasicFormatFile.swift

@@ -151,14 +151,19 @@ let basicFormatFile = SourceFile {
     }
 
     FunctionDecl("open func requiresTrailingSpace(_ token: TokenSyntax) -> Bool") {
-      IfStmt("""
-        // Format `[:]` as-is.
-        if token.tokenKind == .colon && token.parent?.kind == .dictionaryExpr {
+        SwitchStmtSyntax("""
+        switch (token.tokenKind, token.parent?.kind) {
+        case (.colon, .dictionaryExpr): // Ensures there is not space in `[:]`
           return false
+        case (.exclamationMark, .tryExpr), // Ensures there is a space in `try! foo`
+             (.postfixQuestionMark, .tryExpr): // Ensures there is a space in `try? foo`
+          return true
+        default:
+          break
         }
         """)
 
-      SwitchStmt("""
+      SwitchStmtSyntax("""
         switch (token.tokenKind, token.nextToken(viewMode: .sourceAccurate)?.tokenKind) {
         case (.keyword(.as), .exclamationMark), // Ensures there is not space in `as!`
              (.keyword(.as), .postfixQuestionMark), // Ensures there is not space in `as?`
@@ -171,7 +176,7 @@ let basicFormatFile = SourceFile {
              (.postfixQuestionMark, .rightParen), // Ensures there is not space in `myOptionalClosure?()`
              (.keyword(.try), .exclamationMark), // Ensures there is not space in `try!`
              (.keyword(.try), .postfixQuestionMark), // Ensures there is not space in `try?`
-             (.binaryOperator, .comma): // Ensures there is no space in @available(*, deprecated)
+             (.binaryOperator, .comma): // Ensures there is no space in `@available(*, deprecated)`
           return false
         default:
           break

Sources/SwiftBasicFormat/generated/BasicFormat.swift

@@ -163,9 +163,14 @@ open class BasicFormat: SyntaxRewriter {
   }
 
   open func requiresTrailingSpace(_ token: TokenSyntax) -> Bool {
-    // Format `[:]` as-is.
-    if token.tokenKind == .colon && token.parent?.kind == .dictionaryExpr {
+    switch (token.tokenKind, token.parent?.kind) {
+    case (.colon, .dictionaryExpr): // Ensures there is not space in `[:]`
       return false
+    case (.exclamationMark, .tryExpr), // Ensures there is a space in `try! foo`
+     (.postfixQuestionMark, .tryExpr): // Ensures there is a space in `try? foo`
+      return true
+    default: 
+      break 
     }
     switch (token.tokenKind, token.nextToken(viewMode: .sourceAccurate)?.tokenKind) {
     case (.keyword(.as), .exclamationMark), // Ensures there is not space in `as!`
@@ -179,7 +184,7 @@ open class BasicFormat: SyntaxRewriter {
      (.postfixQuestionMark, .rightParen), // Ensures there is not space in `myOptionalClosure?()`
      (.keyword(.try), .exclamationMark), // Ensures there is not space in `try!`
      (.keyword(.try), .postfixQuestionMark), // Ensures there is not space in `try?`
-     (.binaryOperator, .comma): // Ensures there is no space in @available(*, deprecated)
+     (.binaryOperator, .comma): // Ensures there is no space in `@available(*, deprecated)`
       return false
     default: 
       break 
@@ -193,10 +198,6 @@ open class BasicFormat: SyntaxRewriter {
       return true
     case .arrow: 
       return true
-    case .exclamationMark: 
-      return true
-    case .postfixQuestionMark: 
-      return true
     case .poundKeyPathKeyword: 
       return true
     case .poundLineKeyword: 

Sources/SwiftSyntax/gyb_generated/SyntaxFactory.swift

@@ -8689,7 +8689,7 @@ public enum SyntaxFactory {
   @available(*, deprecated, message: "Use TokenSyntax.exclamationMarkToken instead")
   public static func makeExclamationMarkToken(
     leadingTrivia: Trivia = [],
-    trailingTrivia: Trivia = .space
+    trailingTrivia: Trivia = []
   ) -> TokenSyntax {
     return makeToken(.exclamationMark, presence: .present,
                      leadingTrivia: leadingTrivia,
@@ -8698,7 +8698,7 @@ public enum SyntaxFactory {
   @available(*, deprecated, message: "Use TokenSyntax.postfixQuestionMarkToken instead")
   public static func makePostfixQuestionMarkToken(
     leadingTrivia: Trivia = [],
-    trailingTrivia: Trivia = .space
+    trailingTrivia: Trivia = []
   ) -> TokenSyntax {
     return makeToken(.postfixQuestionMark, presence: .present,
                      leadingTrivia: leadingTrivia,

Tests/SwiftSyntaxBuilderTest/FunctionTests.swift

@@ -151,6 +151,18 @@ final class FunctionTests: XCTestCase {
         }
         """
       ),
+      #line: (
+        FunctionDecl(
+          """
+          public func foo(myOptionalValue: String?, myOtherOptionalValue: [String?])  {
+          }
+          """
+        ),
+        """
+        public func foo(myOptionalValue: String?, myOtherOptionalValue: [String?])  {
+        }
+        """
+      ),
       #line: (
         FunctionDeclSyntax(
           modifiers: [DeclModifier(name: .keyword(.public)), DeclModifier(name: .keyword(.static))],

Tests/SwiftSyntaxBuilderTest/VariableTests.swift

@@ -47,6 +47,48 @@ final class VariableTests: XCTestCase {
         }
         """
       ),
+      #line: (
+        VariableDecl("var foo: String? { myOptional?.someProperty }"),
+        """
+        var foo: String? {
+            myOptional?.someProperty
+        }
+        """
+      ),
+      #line: (
+        VariableDecl("let absoluteRaw = AbsoluteRawSyntax(raw: raw!, info: info)"),
+        """
+        let absoluteRaw = AbsoluteRawSyntax(raw: raw!, info: info)
+        """
+      ),
+      #line: (
+        VariableDecl("var foo: String { bar(baz!) }"),
+        """
+        var foo: String {
+            bar(baz!)
+        }
+        """
+      ),
+      #line: (
+        VariableDecl(#"var foo: String { bar ?? "" }"#),
+        #"""
+        var foo: String {
+            bar ?? ""
+        }
+        """#
+      ),
+      #line: (
+        VariableDecl("let bar = try! (foo())"),
+        """
+        let bar = try! (foo())
+        """
+      ),
+      #line: (
+        VariableDecl("let bar = try! !functionThatThrows()"),
+        """
+        let bar = try! !functionThatThrows()
+        """
+      ),
     ]
 
     for (line, testCase) in testCases {

gyb_syntax_support/Token.py

@@ -215,11 +215,9 @@ def macro_name(self):
 
     Punctuator('Backslash', 'backslash', text='\\\\'),
 
-    Punctuator('ExclamationMark', 'exclaim_postfix', text='!',
-               requires_trailing_space=True),
+    Punctuator('ExclamationMark', 'exclaim_postfix', text='!'),
 
-    Punctuator('PostfixQuestionMark', 'question_postfix', text='?',
-               requires_trailing_space=True),
+    Punctuator('PostfixQuestionMark', 'question_postfix', text='?'),
     Punctuator('InfixQuestionMark', 'question_infix', text='?'),
 
     Punctuator('StringQuote', 'string_quote', text='\\\"',