fix: only check once for patch options

This prevents checking for the same patches options multiple times when it is already determined to not have any options
j-hc · Aug 24, 2023 · 11c3a6c · 11c3a6c
1 parent a5851f0
commit 11c3a6c
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/src/main/kotlin/app/revanced/utils/Options.kt b/src/main/kotlin/app/revanced/utils/Options.kt
@@ -62,14 +62,17 @@ internal object Options {
 
             val patchOptions = deserialize(json)
 
-            patches.forEach { patch ->
+            patches.forEach patch@{ patch ->
                 patchOptions.find { option -> option.patchName == patch.patchName }?.let {
                     it.options.forEach { option ->
                         try {
                             patch.options?.set(option.key, option.value)
-                                ?: logger.warning("${patch.patchName} has no options")
+                                ?: run{
+                                    logger.warning("${patch.patchName} has no options")
+                                    return@patch
+                                }
                         } catch (e: NoSuchOptionException) {
-                            logger.info(e.message ?: "Unknown error")
+                            logger.info(e.toString())
                         }
                     }
                 }