Refactor Parser to replace drop(1) with an explicit loop for merging operations.

DmitryNekrasov · DmitryNekrasov · commit abdf1d41c781 · 2025-11-28T09:42:12.000+04:00
diff --git a/core/common/src/internal/format/parser/Parser.kt b/core/common/src/internal/format/parser/Parser.kt
@@ -58,7 +58,9 @@ internal fun <T> List<ParserStructure<T>>.concat(): ParserStructure<T> {
                 }
                 firstOperation is NumberSpanParserOperation -> {
                     add(NumberSpanParserOperation(numberSpan + firstOperation.consumers))
-                    addAll(operationsToMerge.drop(1))
+                    for (i in 1..operationsToMerge.lastIndex) {
+                        add(operationsToMerge[i])
+                    }
                 }
                 else -> {
                     add(NumberSpanParserOperation(numberSpan))

Original file line number	Diff line number	Diff line change
`@@ -58,7 +58,9 @@ internal fun <T> List<ParserStructure<T>>.concat(): ParserStructure<T> {`
`58`	`58`	`}`
`59`	`59`	`firstOperation is NumberSpanParserOperation -> {`
`60`	`60`	`add(NumberSpanParserOperation(numberSpan + firstOperation.consumers))`
`61`		`- addAll(operationsToMerge.drop(1))`
	`61`	`+ for (i in 1..operationsToMerge.lastIndex) {`
	`62`	`+ add(operationsToMerge[i])`
	`63`	`+ }`
`62`	`64`	`}`
`63`	`65`	`else -> {`
`64`	`66`	`add(NumberSpanParserOperation(numberSpan))`