perf: Improve proxy performance

[oSumAtrIX]

Feature description

Right now, the proxy function is implemented like this:

Make proxyPool a map for constant lookup.

Motivation

Proxy has linear time complexity and if you need to proxy many classes, it's quadratic. Making lookup constant would reduce complexity to linear for performance.

Acknowledgements

• I have checked all open and closed feature requests and this is not a duplicate.
• I have chosen an appropriate title.
• All requested information has been provided properly.

[oSumAtrIX]

@LisoUseInAIKyrios Does that make sense?

[LisoUseInAIKyrios]

This is already implemented in

#329 (comment)

https://github.com/LisoUseInAIKyrios/revanced-patcher/blob/feat/instruction_filters/src/main/kotlin/app/revanced/patcher/util/PatchClasses.kt

[LisoUseInAIKyrios]

The change also fixes an existing bug, where if a patch modified a method, then a different patch fingerprint matches a different part of the same method, then the match indexes returned by the second fingerprint are incorrect (it returns the pattern indexes of the unmodified method).

#329 (comment)

The bug has existed for a long time, but that change fixes it.

[oSumAtrIX]

The bug has existed for a long time, but that change fixes it.

It was considered a non-bug. I was under the impression of fingerprints matching the real source, but it makes sense to make it respect past changes.