Merge pull request #2183 from apple/revert-2150-kimdv/2147-sometimes-the-placeholder-is-identified-as-identifierpattern

Revert "Handle pattern as editor placeholder"

Author: kimdv

CodeGeneration/Sources/SyntaxSupport/PatternNodes.swift

@@ -182,22 +182,4 @@ public let PATTERN_NODES: [Node] = [
     ]
   ),
 
-  Node(
-    kind: .editorPlaceholderPattern,
-    base: .pattern,
-    nameForDiagnostics: "editor placeholder",
-    documentation: """
-      An editor placeholder, e.g. `<#pattern#>` that is used in a position that expects a pattern.
-      """,
-    children: [
-      Child(
-        name: "placeholder",
-        kind: .token(choices: [.token(.identifier)]),
-        documentation: """
-          The actual editor placeholder that starts with `<#` and ends with `#>`.
-          """
-      )
-    ]
-  ),
-
 ]

CodeGeneration/Sources/SyntaxSupport/SyntaxNodeKind.swift

@@ -111,8 +111,6 @@ public enum SyntaxNodeKind: String, CaseIterable {
   case dynamicReplacementAttributeArguments
   case editorPlaceholderDecl
   case editorPlaceholderExpr
-  case editorPlaceholderPattern
-  case editorPlaceholderType
   case effectsAttributeArgumentList
   case enumCaseDecl
   case enumCaseElement

CodeGeneration/Sources/SyntaxSupport/TypeNodes.swift

@@ -530,28 +530,4 @@ public let TYPE_NODES: [Node] = [
     ]
   ),
 
-  Node(
-    kind: .editorPlaceholderType,
-    base: .type,
-    nameForDiagnostics: "editor placeholder",
-    documentation: """
-      An editor placeholder, e.g. `<#Type#>` that is used in a position that expects a type.
-      """,
-    children: [
-      Child(
-        name: "placeholder",
-        kind: .token(choices: [.token(.identifier)]),
-        documentation: """
-          The actual editor placeholder that starts with `<#` and ends with `#>`.
-          """
-      ),
-      Child(
-        name: "genericArgumentClause",
-        kind: .node(kind: .genericArgumentClause),
-        documentation: "Generic arguments that is attatched to the type.",
-        isOptional: true
-      ),
-    ]
-  ),
-
 ]

Release Notes/510.md

@@ -21,16 +21,6 @@
   - Description: Remarks are used by the Swift compiler and other tools to describe some aspect of translation that doesn't reflect correctness, but may be useful for the user. Remarks have been added to the diagnostic severity enums to align with the Swift compiler.
   - Pull Request: https://github.com/apple/swift-syntax/pull/2143
 
-- New pattern node `EditorPlaceholderPatternSyntax`
-  - Description: This node type will be placeholder that is used in a position that expects a pattern.
-  - Issue: https://github.com/apple/swift-syntax/issues/2147
-  - Pull Request: https://github.com/apple/swift-syntax/pull/2150
-
-- New type node `EditorPlaceholderTypeSyntax`
-  - Description: This node type will be placeholder that is used in a position that expects a type.
-  - Issue: https://github.com/apple/swift-syntax/issues/2156
-  - Pull Request: https://github.com/apple/swift-syntax/pull/2150
-
 ## API Behavior Changes
 
 ## Deprecations

Sources/SwiftParser/Patterns.swift

@@ -70,21 +70,12 @@ extension Parser {
       )
     case (.lhs(.identifier), let handle)?:
       let identifier = self.eat(handle)
-      if identifier.tokenText.isEditorPlaceholder {
-        return RawPatternSyntax(
-          RawEditorPlaceholderPatternSyntax(
-            placeholder: identifier,
-            arena: self.arena
-          )
-        )
-      } else {
-        return RawPatternSyntax(
-          RawIdentifierPatternSyntax(
-            identifier: identifier,
-            arena: self.arena
-          )
+      return RawPatternSyntax(
+        RawIdentifierPatternSyntax(
+          identifier: identifier,
+          arena: self.arena
         )
-      }
+      )
     case (.lhs(.dollarIdentifier), let handle)?:
       let dollarIdent = self.eat(handle)
       let unexpectedBeforeIdentifier = RawUnexpectedNodesSyntax(elements: [RawSyntax(dollarIdent)], arena: self.arena)

Sources/SwiftParser/Types.swift

@@ -349,25 +349,14 @@ extension Parser {
       generics = nil
     }
 
-    if name.tokenText.isEditorPlaceholder {
-      return RawTypeSyntax(
-        RawEditorPlaceholderTypeSyntax(
-          placeholder: name,
-          genericArgumentClause: generics,
-          arena: self.arena
-        )
-      )
-    } else {
-      return RawTypeSyntax(
-        RawIdentifierTypeSyntax(
-          unexpectedBeforeName,
-          name: name,
-          genericArgumentClause: generics,
-          arena: self.arena
-        )
+    return RawTypeSyntax(
+      RawIdentifierTypeSyntax(
+        unexpectedBeforeName,
+        name: name,
+        genericArgumentClause: generics,
+        arena: self.arena
       )
-    }
-
+    )
   }
 
   /// Parse the existential `Any` type.
@@ -949,9 +938,7 @@ extension Parser {
 
 extension Parser {
   mutating func parseResultType() -> RawTypeSyntax {
-    if self.at(prefix: "<#") {
-      return self.parseTypeIdentifier()
-    } else if self.at(prefix: "<") {
+    if self.at(prefix: "<") {
       let generics = self.parseGenericParameters()
       let baseType = self.parseType()
       return RawTypeSyntax(

Sources/SwiftParserDiagnostics/generated/SyntaxKindNameForDiagnostics.swift

@@ -143,10 +143,6 @@ extension SyntaxKind {
       return "editor placeholder"
     case .editorPlaceholderExpr:
       return "editor placeholder"
-    case .editorPlaceholderPattern:
-      return "editor placeholder"
-    case .editorPlaceholderType:
-      return "editor placeholder"
     case .effectsAttributeArgumentList:
       return "@_effects arguments"
     case .enumCaseDecl:

Sources/SwiftSyntax/Documentation.docc/generated/SwiftSyntax.md

@@ -151,7 +151,6 @@ These articles are intended for developers wishing to contribute to SwiftSyntax
 - <doc:SwiftSyntax/PatternSyntax>
 - <doc:SwiftSyntax/PatternSyntaxProtocol>
 - <doc:SwiftSyntax/MissingPatternSyntax>
-- <doc:SwiftSyntax/EditorPlaceholderPatternSyntax>
 - <doc:SwiftSyntax/ExpressionPatternSyntax>
 - <doc:SwiftSyntax/IdentifierPatternSyntax>
 - <doc:SwiftSyntax/IsTypePatternSyntax>
@@ -191,7 +190,6 @@ These articles are intended for developers wishing to contribute to SwiftSyntax
 - <doc:SwiftSyntax/ClassRestrictionTypeSyntax>
 - <doc:SwiftSyntax/CompositionTypeSyntax>
 - <doc:SwiftSyntax/DictionaryTypeSyntax>
-- <doc:SwiftSyntax/EditorPlaceholderTypeSyntax>
 - <doc:SwiftSyntax/FunctionTypeSyntax>
 - <doc:SwiftSyntax/IdentifierTypeSyntax>
 - <doc:SwiftSyntax/ImplicitlyUnwrappedOptionalTypeSyntax>

Sources/SwiftSyntax/generated/ChildNameForKeyPath.swift

@@ -1077,22 +1077,6 @@ public func childName(_ keyPath: AnyKeyPath) -> String? {
     return "placeholder"
   case \EditorPlaceholderExprSyntax.unexpectedAfterPlaceholder:
     return "unexpectedAfterPlaceholder"
-  case \EditorPlaceholderPatternSyntax.unexpectedBeforePlaceholder:
-    return "unexpectedBeforePlaceholder"
-  case \EditorPlaceholderPatternSyntax.placeholder:
-    return "placeholder"
-  case \EditorPlaceholderPatternSyntax.unexpectedAfterPlaceholder:
-    return "unexpectedAfterPlaceholder"
-  case \EditorPlaceholderTypeSyntax.unexpectedBeforePlaceholder:
-    return "unexpectedBeforePlaceholder"
-  case \EditorPlaceholderTypeSyntax.placeholder:
-    return "placeholder"
-  case \EditorPlaceholderTypeSyntax.unexpectedBetweenPlaceholderAndGenericArgumentClause:
-    return "unexpectedBetweenPlaceholderAndGenericArgumentClause"
-  case \EditorPlaceholderTypeSyntax.genericArgumentClause:
-    return "genericArgumentClause"
-  case \EditorPlaceholderTypeSyntax.unexpectedAfterGenericArgumentClause:
-    return "unexpectedAfterGenericArgumentClause"
   case \EnumCaseDeclSyntax.unexpectedBeforeAttributes:
     return "unexpectedBeforeAttributes"
   case \EnumCaseDeclSyntax.attributes:

Sources/SwiftSyntax/generated/SyntaxAnyVisitor.swift

@@ -776,22 +776,6 @@ open class SyntaxAnyVisitor: SyntaxVisitor {
     visitAnyPost(node._syntaxNode)
   }
 
-  override open func visit(_ node: EditorPlaceholderPatternSyntax) -> SyntaxVisitorContinueKind {
-    return visitAny(node._syntaxNode)
-  }
-  
-  override open func visitPost(_ node: EditorPlaceholderPatternSyntax) {
-    visitAnyPost(node._syntaxNode)
-  }
-  
-  override open func visit(_ node: EditorPlaceholderTypeSyntax) -> SyntaxVisitorContinueKind {
-    return visitAny(node._syntaxNode)
-  }
-  
-  override open func visitPost(_ node: EditorPlaceholderTypeSyntax) {
-    visitAnyPost(node._syntaxNode)
-  }
-  
   override open func visit(_ node: EffectsAttributeArgumentListSyntax) -> SyntaxVisitorContinueKind {
     return visitAny(node._syntaxNode)
   }