Prefix should match when maxLength < match.count (#50)

* Prefix should match when maxLength < match.count

* cleanup

* wip

Package.resolved

@@ -11,7 +11,7 @@
         }
       },
       {
-        "package": "swift-benchmark",
+        "package": "Benchmark",
         "repositoryURL": "https://github.com/google/swift-benchmark",
         "state": {
           "branch": null,

Sources/Parsing/Parsers/Prefix.swift

@@ -148,19 +148,12 @@ where
 
   @inlinable
   public func parse(_ input: inout Input) -> Input? {
-    if let predicate = self.predicate {
-      let prefix = input.prefix(while: predicate)
-      let count = prefix.count
-      guard count >= self.minLength, self.maxLength.map({ count <= $0 }) ?? true else { return nil }
-      input.removeFirst(count)
-      return prefix
-    } else {
-      let prefix = self.maxLength.map(input.prefix) ?? input
-      let count = prefix.count
-      guard count >= self.minLength else { return nil }
-      input.removeFirst(count)
-      return prefix
-    }
+    var prefix = maxLength.map(input.prefix) ?? input
+    prefix = predicate.map { prefix.prefix(while: $0) } ?? prefix
+    let count = prefix.count
+    guard count >= self.minLength else { return nil }
+    input.removeFirst(count)
+    return prefix
   }
 }
 

Tests/ParsingTests/PrefixTests.swift

@@ -61,4 +61,10 @@ final class PrefixTests: XCTestCase {
     XCTAssertEqual("42", Prefix(...10, while: { $0.isNumber }).parse(&input))
     XCTAssertEqual(" Hello, world!", input)
   }
+
+  func testPrefixLengthFromWhileSuccess() {
+    var input = "42 Hello, world!"[...]
+    XCTAssertEqual("4", Prefix(1, while: { $0.isNumber }).parse(&input))
+    XCTAssertEqual("2 Hello, world!", input)
+  }
 }