fix: remove unnecessary dummy nop instructions (#111)

This commit is contained in:
MewtR 2022-11-14 10:57:13 -05:00 committed by GitHub
parent d2f91a8545
commit f9bc95f220
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -139,10 +139,10 @@ fun MutableMethod.addInstructions(index: Int, smali: String, externalLabels: Lis
// Add the compiled list of instructions to the method. // Add the compiled list of instructions to the method.
val methodImplementation = this.implementation!! val methodImplementation = this.implementation!!
methodImplementation.addInstructions(index, compiledInstructions) methodImplementation.addInstructions(index, compiledInstructions.subList(0, compiledInstructions.size - externalLabels.size))
val methodInstructions = methodImplementation.instructions val methodInstructions = methodImplementation.instructions
methodInstructions.subList(index, index + compiledInstructions.size) methodInstructions.subList(index, index + compiledInstructions.size - externalLabels.size)
.forEachIndexed { compiledInstructionIndex, compiledInstruction -> .forEachIndexed { compiledInstructionIndex, compiledInstruction ->
// If the compiled instruction is not an offset instruction, skip it. // If the compiled instruction is not an offset instruction, skip it.
if (compiledInstruction !is BuilderOffsetInstruction) return@forEachIndexed if (compiledInstruction !is BuilderOffsetInstruction) return@forEachIndexed