cleanup

Author: milseman

Sources/_StringProcessing/Engine/MEBuiltins.swift

@@ -30,6 +30,7 @@ extension Processor {
   }
 
   func isAtStartOfLine(_ payload: AssertionPayload) -> Bool {
+    // TODO: needs benchmark coverage
     if currentPosition == subjectBounds.lowerBound { return true }
     switch payload.semanticLevel {
     case .graphemeCluster:
@@ -40,6 +41,7 @@ extension Processor {
   }
 
   func isAtEndOfLine(_ payload: AssertionPayload) -> Bool {
+    // TODO: needs benchmark coverage
     if currentPosition == subjectBounds.upperBound { return true }
     switch payload.semanticLevel {
     case .graphemeCluster:
@@ -50,6 +52,8 @@ extension Processor {
   }
 
   mutating func builtinAssert(by payload: AssertionPayload) throws -> Bool {
+    // TODO: needs benchmark coverage
+
     // Future work: Optimize layout and dispatch
     switch payload.kind {
     case .startOfSubject: return currentPosition == subjectBounds.lowerBound

Sources/_StringProcessing/Engine/MEQuantify.swift

@@ -20,6 +20,7 @@ extension Processor {
         isStrictASCII: payload.builtinIsStrict,
         isScalarSemantics: false)
     case .any:
+      // TODO: call out to existing code with quick check
       let matched = currentPosition != input.endIndex
         && (!input[currentPosition].isNewline || payload.anyMatchesNewline)
       next = matched ? input.index(after: currentPosition) : nil

Sources/_StringProcessing/Engine/Processor.swift

@@ -686,22 +686,8 @@ extension Processor {
       controller.step()
     }
   }
-
-  func sleep() {
-    var i = 0
-    for c in input {
-      if i > 20 { break }
-      i += 1
-      if c == "C" {
-        blackHole(c)
-      }
-    }
-  }
 }
 
-@inline(never)
-func blackHole<T>(_ t: T) { _ = t }
-
 extension String {
 
   func match(
@@ -740,10 +726,6 @@ extension String {
     return cur
   }
 
-  // func consumeScalar(_ n: Distance) -> Bool {
-
-  // }
-
   func matchScalar(
     _ scalar: Unicode.Scalar,
     at pos: Index,