From f10f5e29102148f75cb157b926227eee87f28ee7 Mon Sep 17 00:00:00 2001 From: oSumAtrIX Date: Sat, 7 Mar 2026 23:39:50 +0100 Subject: [PATCH] refactor(YouTube - Add more double tap to seek length options): Use more idiomatic code --- .../AddMoreDoubleTapToSeekLengthOptionsPatch.kt | 14 ++++---------- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/patches/src/main/kotlin/app/revanced/patches/youtube/interaction/doubletap/AddMoreDoubleTapToSeekLengthOptionsPatch.kt b/patches/src/main/kotlin/app/revanced/patches/youtube/interaction/doubletap/AddMoreDoubleTapToSeekLengthOptionsPatch.kt index b84afcb4d9..c2b1686567 100644 --- a/patches/src/main/kotlin/app/revanced/patches/youtube/interaction/doubletap/AddMoreDoubleTapToSeekLengthOptionsPatch.kt +++ b/patches/src/main/kotlin/app/revanced/patches/youtube/interaction/doubletap/AddMoreDoubleTapToSeekLengthOptionsPatch.kt @@ -28,12 +28,7 @@ val addMoreDoubleTapToSeekLengthOptionsPatch = resourcePatch( execute { // Values are hard coded to keep patching simple. - val doubleTapLengthOptionsString = "3, 5, 10, 15, 20, 30, 60, 120, 180, 240" - - val doubleTapLengths = doubleTapLengthOptionsString - .replace(" ", "") - .split(",") - if (doubleTapLengths.isEmpty()) throw PatchException("Invalid double-tap length elements") + val doubleTapLengths = listOf(3, 5, 10, 15, 20, 30, 60, 120, 180, 240) document("res/values/arrays.xml").use { document -> fun Element.removeAllChildren() { @@ -56,10 +51,9 @@ val addMoreDoubleTapToSeekLengthOptionsPatch = resourcePatch( entries.removeAllChildren() doubleTapLengths.forEach { length -> - val item = document.createElement("item") - item.textContent = length - entries.appendChild(item) - values.appendChild(item.cloneNode(true)) + document.createElement("item").apply { textContent = length.toString() } + .also(entries::appendChild) + .cloneNode(true).let(values::appendChild) } } }