external/revanced-patcher
1
0
Fork 0
mirror of https://github.com/ReVanced/revanced-patcher.git synced 2024-11-10 09:08:04 +01:00
Code Issues Projects Releases Packages Wiki Activity

fix: ConcurrentModificationException while iterating through proxies and modifying it

Browse source 
Signed-off-by: oSumAtrIX <johan.melkonyan1@web.de>
This commit is contained in:
oSumAtrIX 2022-04-19 20:07:31 +02:00
parent beff1df9b0
commit 6cb7cdb0b2
No known key found for this signature in database
GPG key ID: A9B3094ACDB604B4
1 changed files with 8 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
src/main/kotlin/app/revanced/patcher/util/ProxyBackedClassList.kt
View file

@ -15,18 +15,21 @@ class ProxyBackedClassList(internal val internalClasses: MutableList<ClassDef>)
}
/**
* Apply all resolved classes into [internalClasses] and clear the [proxies] list.
* Apply all resolved classes into [internalClasses] and clean the [proxies] list.
*/
fun applyProxies() {
proxies.forEachIndexed { i, proxy ->
if (!proxy.proxyUsed) return@forEachIndexed
// FIXME: check if this could cause issues when multiple patches use the same proxy
proxies.removeIf { proxy ->
// if the proxy is unused, keep it in the list
if (!proxy.proxyUsed) return@removeIf false
// if it has been used, replace the internal class which it proxied
val index = internalClasses.indexOfFirst { it.type == proxy.immutableClass.type }
internalClasses[index] = proxy.mutatedClass
proxies.removeAt(i) // FIXME: check if this could cause issues when multiple patches use the same proxy
// return true to remove it from the proxies list
return@removeIf true
}
}
override val size get() = internalClasses.size