Revert "Merge pull request swiftlang#628 from apple/result_builder_changes_workaround"

This reverts commit 7e059b7, reversing
changes made to 3ca8b13.

Author: natecook1000

Sources/_StringProcessing/MatchingOptions.swift

@@ -14,9 +14,7 @@
 /// A type that represents the current state of regex matching options, with
 /// stack-based scoping.
 struct MatchingOptions {
-  // FIXME: Workaround for rdar://104923020
-  // fileprivate
-  var stack: [Representation]
+  fileprivate var stack: [Representation]
 
   fileprivate func _invariantCheck() {
     assert(!stack.isEmpty, "Unbalanced call to endScope")
@@ -127,9 +125,7 @@ extension MatchingOptions {
 // MARK: - Implementation
 extension MatchingOptions {
   /// An option that changes the behavior of a regular expression.
-  // FIXME: Workaround for rdar://104923020
-  // fileprivate
-  enum Option: Int {
+  fileprivate enum Option: Int {
     // PCRE options
     case caseInsensitive
     case allowDuplicateGroupNames
@@ -216,9 +212,7 @@ extension MatchingOptions {
 
 extension MatchingOptions {
   /// A set of matching options.
-  // FIXME: Workaround for rdar://104923020
-  // fileprivate
-  struct Representation: OptionSet, RawRepresentable {
+  fileprivate struct Representation: OptionSet, RawRepresentable {
     var rawValue: UInt32
 
     /// Returns `true` if the option denoted by `kind` is a member of this set.

Tests/RegexBuilderTests/CustomTests.swift

@@ -226,7 +226,7 @@ class CustomRegexComponentTests: XCTestCase {
   // tests.
   func testCustomRegexComponents() throws {
     customTest(
-      Regex<Substring> {
+      Regex {
         Numbler()
         Asciibbler()
       },
@@ -237,7 +237,7 @@ class CustomRegexComponentTests: XCTestCase {
       ("t4", .match, nil))
 
     customTest(
-      Regex<Substring> {
+      Regex {
         OneOrMore { Numbler() }
       },
       ("ab123c", .firstMatch, "123"),
@@ -520,7 +520,7 @@ class CustomRegexComponentTests: XCTestCase {
     )
 
     customTest(
-      Regex<CurrencyParser.Currency> {
+      Regex {
         CurrencyParser()
       },
       ("USD", .usd, nil),
@@ -532,7 +532,7 @@ class CustomRegexComponentTests: XCTestCase {
 
     // No capture, two errors
     customTest(
-      Regex<Substring> {
+      Regex {
         IntParser()
         " "
         IntParser()
@@ -544,7 +544,7 @@ class CustomRegexComponentTests: XCTestCase {
     )
 
     customTest(
-      Regex<Substring> {
+      Regex {
         CurrencyParser()
         IntParser()
       },
@@ -557,7 +557,7 @@ class CustomRegexComponentTests: XCTestCase {
     // One capture, two errors: One error is thrown from inside a capture,
     // while the other one is thrown from outside
     customTest(
-      Regex<(Substring, CurrencyParser.Currency)> {
+      Regex {
         Capture { CurrencyParser() }
         IntParser()
       },
@@ -568,7 +568,7 @@ class CustomRegexComponentTests: XCTestCase {
     )
 
     customTest(
-      Regex<(Substring, Int)> {
+      Regex {
         CurrencyParser()
         Capture { IntParser() }
       },
@@ -580,7 +580,7 @@ class CustomRegexComponentTests: XCTestCase {
 
     // One capture, two errors: Both errors are thrown from inside the capture
     customTest(
-      Regex<(Substring, Substring)> {
+      Regex {
         Capture {
           CurrencyParser()
           IntParser()
@@ -594,7 +594,7 @@ class CustomRegexComponentTests: XCTestCase {
 
     // Two captures, two errors: Different erros are thrown from inside captures
     customTest(
-      Regex<(Substring, CurrencyParser.Currency, Int)> {
+      Regex {
         Capture(CurrencyParser())
         Capture(IntParser())
       },

Tests/RegexBuilderTests/MotivationTests.swift

@@ -310,8 +310,8 @@ extension RegexDSLTests {
           "CREDIT"
           "DEBIT"
         }
-      } transform: { (s: Substring) -> TransactionKind? in
-        TransactionKind(rawValue: String(s))
+      } transform: {
+        TransactionKind(rawValue: String($0))
       }
 
       OneOrMore(.whitespace)
@@ -323,8 +323,8 @@ extension RegexDSLTests {
         Repeat(.digit, count: 2)
         Repeat(.digit, count: 2)
         Repeat(.digit, count: 4)
-      } transform: { (s: Substring) -> Date? in
-        Date(mmddyyyy: String(s))
+      } transform: {
+        Date(mmddyyyy: String($0))
       }
 
       OneOrMore(.whitespace)
@@ -346,8 +346,8 @@ extension RegexDSLTests {
         OneOrMore(.digit)
         "."
         Repeat(.digit, count: 2)
-      } transform: { (s: Substring) -> Double? in
-        Double(s)
+      } transform: {
+        Double($0)
       }
     }
 

Tests/RegexBuilderTests/RegexDSLTests.swift

@@ -1254,8 +1254,8 @@ class RegexDSLTests: XCTestCase {
         TryCapture(as: b) {
           "#"
           OneOrMore(.digit)
-        } transform: { (s: Substring) in
-          Int(s.dropFirst())
+        } transform: {
+          Int($0.dropFirst())
         }
       }
       a
@@ -1272,14 +1272,14 @@ class RegexDSLTests: XCTestCase {
     do {
       let a = Reference(Substring.self)
       let b = Reference(Int.self)
-      let regex = Regex<(Substring, Substring, Int?, Int?, Substring?)> {
+      let regex = Regex {
         Capture("abc", as: a)
         ZeroOrMore {
           TryCapture(as: b) {
             "#"
             OneOrMore(.digit)
-          } transform: { (s: Substring) -> Int? in
-            Int(s.dropFirst())
+          } transform: {
+            Int($0.dropFirst())
           }
         }
         a