Address review comments

Catfish-Man · Catfish-Man · commit 93a220bff3de · 2024-02-01T14:44:08.000-08:00
diff --git a/Sources/_StringProcessing/Engine/MEQuantify.swift b/Sources/_StringProcessing/Engine/MEQuantify.swift
@@ -130,6 +130,7 @@ extension Processor {
       return false
     }
     if let savePointRange {
+      assert(!produceSavePointRange)
       savePoints.append(makeQuantifiedSavePoint(
         savePointRange, isScalarSemantics: payload.isScalarSemantics))
     }
@@ -203,7 +204,10 @@ extension String {
     // one position, because newline-sequence in scalar semantic mode still
     // matches two scalars
 
-    return (currentPosition, rangeStart..<rangeEnd)
+    return (
+      currentPosition,
+      Range(uncheckedBounds: (lower: rangeStart, upper: rangeEnd))
+    )
   }
 
   // NOTE: [Zero|One]OrMore overloads are to specialize the inlined run loop,

Original file line number	Diff line number	Diff line change
`@@ -130,6 +130,7 @@ extension Processor {`
`130`	`130`	`return false`
`131`	`131`	`}`
`132`	`132`	`if let savePointRange {`
	`133`	`+ assert(!produceSavePointRange)`
`133`	`134`	`savePoints.append(makeQuantifiedSavePoint(`
`134`	`135`	`savePointRange, isScalarSemantics: payload.isScalarSemantics))`
`135`	`136`	`}`
`@@ -203,7 +204,10 @@ extension String {`
`203`	`204`	`// one position, because newline-sequence in scalar semantic mode still`
`204`	`205`	`// matches two scalars`
`205`	`206`
`206`		`- return (currentPosition, rangeStart..<rangeEnd)`
	`207`	`+ return (`
	`208`	`+ currentPosition,`
	`209`	`+ Range(uncheckedBounds: (lower: rangeStart, upper: rangeEnd))`
	`210`	`+ )`
`207`	`211`	`}`
`208`	`212`
`209`	`213`	`// NOTE: [Zero\|One]OrMore overloads are to specialize the inlined run loop,`