perf(fingerprint): do not resolve already resolved fingerprints

Signed-off-by: oSumAtrIX <johan.melkonyan1@web.de>
tathaha · Sep 18, 2022 · 4bfd7eb · 4bfd7eb
1 parent 2f7e62e
commit 4bfd7eb
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/main/kotlin/app/revanced/patcher/fingerprint/method/utils/MethodFingerprintUtils.kt b/src/main/kotlin/app/revanced/patcher/fingerprint/method/utils/MethodFingerprintUtils.kt
@@ -50,11 +50,13 @@ object MethodFingerprintUtils {
      * @param context The context on which to resolve the [MethodFingerprint].
      * @param classDef The class of the matching [Method].
      * @param forData The [BytecodeData] to host proxies.
-     * @return True if the resolution was successful, false otherwise.
+     * @return True if the resolution was successful or if the fingerprint is already resolved, false otherwise.
      */
     fun MethodFingerprint.resolve(forData: BytecodeData, context: Method, classDef: ClassDef): Boolean {
         val methodFingerprint = this
 
+        if (methodFingerprint.result != null) return true
+
         if (methodFingerprint.returnType != null && !context.returnType.startsWith(methodFingerprint.returnType))
             return false