Fix a crash that calls dropLast with a negative number

intitni · intitni · commit 93c54046d0f1 · 2023-11-09T17:04:03.000+08:00
diff --git a/Tool/Sources/SuggestionModel/EditorInformation.swift b/Tool/Sources/SuggestionModel/EditorInformation.swift
@@ -102,12 +102,12 @@ public struct EditorInformation {
         }
         var content = rangeLines
         if !content.isEmpty {
+            let dropLastCount = max(0, content[content.endIndex - 1].count - range.end.character)
             content[content.endIndex - 1] = String(
-                content[content.endIndex - 1].dropLast(
-                    content[content.endIndex - 1].count - range.end.character
-                )
+                content[content.endIndex - 1].dropLast(dropLastCount)
             )
-            content[0] = String(content[0].dropFirst(range.start.character))
+            let dropFirstCount = max(0, range.start.character)
+            content[0] = String(content[0].dropFirst(dropFirstCount))
         }
         return (content.joined(), rangeLines)
     }

Original file line number	Diff line number	Diff line change
`@@ -102,12 +102,12 @@ public struct EditorInformation {`
`102`	`102`	`}`
`103`	`103`	`var content = rangeLines`
`104`	`104`	`if !content.isEmpty {`
	`105`	`+ let dropLastCount = max(0, content[content.endIndex - 1].count - range.end.character)`
`105`	`106`	`content[content.endIndex - 1] = String(`
`106`		`- content[content.endIndex - 1].dropLast(`
`107`		`- content[content.endIndex - 1].count - range.end.character`
`108`		`- )`
	`107`	`+ content[content.endIndex - 1].dropLast(dropLastCount)`
`109`	`108`	`)`
`110`		`- content[0] = String(content[0].dropFirst(range.start.character))`
	`109`	`+ let dropFirstCount = max(0, range.start.character)`
	`110`	`+ content[0] = String(content[0].dropFirst(dropFirstCount))`
`111`	`111`	`}`
`112`	`112`	`return (content.joined(), rangeLines)`
`113`	`113`	`}`