diff --git a/js/js.libraries/src/core/string.kt b/js/js.libraries/src/core/string.kt index 90e9a06f2e0..ac110beeb00 100644 --- a/js/js.libraries/src/core/string.kt +++ b/js/js.libraries/src/core/string.kt @@ -26,6 +26,14 @@ deprecated("Use split(Regex) instead.") library("splitString") public fun String.splitWithRegex(regex: String, limit: Int): Array = noImpl +deprecated("Use split(Regex) instead.") +library("splitString") +public fun String.split(regex: String): Array = noImpl + +deprecated("Use split(Regex) instead.") +library("splitString") +public fun String.split(regex: String, limit: Int): Array = noImpl + native public fun String.substring(beginIndex : Int) : String = noImpl native public fun String.substring(beginIndex : Int, endIndex : Int) : String = noImpl diff --git a/libraries/stdlib/src/kotlin/text/Strings.kt b/libraries/stdlib/src/kotlin/text/Strings.kt index 410c974ae7a..b673fd56f6f 100644 --- a/libraries/stdlib/src/kotlin/text/Strings.kt +++ b/libraries/stdlib/src/kotlin/text/Strings.kt @@ -922,10 +922,6 @@ public fun String.splitToSequence(vararg delimiters: String, ignoreCase: Boolean * the beginning to the end of this string, and matches at each position the first element in [delimiters] * that is equal to a delimiter in this instance at that position. */ -public fun String.split(vararg delimiters: String, ignoreCase: Boolean = false, limit: Int = 0): List = - splitToSequence(*delimiters, ignoreCase = ignoreCase, limit = limit).toList() - -deprecated("Use split(delimiters) instead.") public fun String.splitBy(vararg delimiters: String, ignoreCase: Boolean = false, limit: Int = 0): List = splitToSequence(*delimiters, ignoreCase = ignoreCase, limit = limit).toList() diff --git a/libraries/stdlib/src/kotlin/text/StringsJVM.kt b/libraries/stdlib/src/kotlin/text/StringsJVM.kt index f5793da5575..c2237f76778 100644 --- a/libraries/stdlib/src/kotlin/text/StringsJVM.kt +++ b/libraries/stdlib/src/kotlin/text/StringsJVM.kt @@ -127,6 +127,12 @@ public fun String.split(regex: Pattern, limit: Int = 0): List require(limit >= 0, { "Limit must be non-negative, but was $limit" } ) return regex.split(this, if (limit == 0) -1 else limit).asList() } +/** + * Splits this string around matches of the given regular expression. + */ +deprecated("Convert an argument to regex with toRegex or use splitBy instead.") +public fun String.split(regex: String): Array = split(regex.toRegex()).toTypedArray() + /** * Returns a substring of this string starting with the specified index. diff --git a/libraries/stdlib/test/text/StringTest.kt b/libraries/stdlib/test/text/StringTest.kt index d55ac51c03d..9e9c9da078c 100644 --- a/libraries/stdlib/test/text/StringTest.kt +++ b/libraries/stdlib/test/text/StringTest.kt @@ -353,18 +353,18 @@ class StringTest { test fun split() { - assertEquals(listOf(""), "".split(";")) + assertEquals(listOf(""), "".splitBy(";")) assertEquals(listOf("test"), "test".split(*charArray()), "empty list of delimiters, none matched -> entire string returned") - assertEquals(listOf("test"), "test".split(*array()), "empty list of delimiters, none matched -> entire string returned") + assertEquals(listOf("test"), "test".splitBy(*array()), "empty list of delimiters, none matched -> entire string returned") assertEquals(listOf("abc", "def", "123;456"), "abc;def,123;456".split(';', ',', limit = 3)) - assertEquals(listOf("abc", "def", "123", "456"), "abc
def
123
456".split("
", ignoreCase = true)) + assertEquals(listOf("abc", "def", "123", "456"), "abc
def
123
456".splitBy("
", ignoreCase = true)) - assertEquals(listOf("abc", "def", "123", "456"), "abc=-def==123=456".split("==", "=-", "=")) + assertEquals(listOf("abc", "def", "123", "456"), "abc=-def==123=456".splitBy("==", "=-", "=")) - assertEquals(listOf("", "a", "b", "c", ""), "abc".split("")) - assertEquals(listOf("", "a", "b", "b", "a", ""), "abba".split("", "a")) - assertEquals(listOf("", "", "b", "b", "", ""), "abba".split("a", "")) + assertEquals(listOf("", "a", "b", "c", ""), "abc".splitBy("")) + assertEquals(listOf("", "a", "b", "b", "a", ""), "abba".splitBy("", "a")) + assertEquals(listOf("", "", "b", "b", "", ""), "abba".splitBy("a", "")) } test fun splitToLines() {