github-repos/testable-mock
1
0
Fork 0
mirror of https://github.com/alibaba/testable-mock.git synced 2025-03-23 16:20:26 +08:00
Code Issues Packages Projects Releases Wiki Activity

local variable table of non-static method can be empty

Browse Source
This commit is contained in:
金戟 2021-04-20 21:17:09 +08:00
parent 31140bef87
commit d463f863db
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
testable-agent/src/main/java/com/alibaba/testable/agent/handler/MockClassHandler.java
View File

@ -101,7 +101,8 @@ public class MockClassHandler extends BaseClassWithContextHandler {
// must get label before method description changed // must get label before method description changed
ImmutablePair<LabelNode, LabelNode> labels = getStartAndEndLabel(mn); ImmutablePair<LabelNode, LabelNode> labels = getStartAndEndLabel(mn);
mn.desc = MethodUtil.addParameterAtBegin(mn.desc, targetClassName); mn.desc = MethodUtil.addParameterAtBegin(mn.desc, targetClassName);
int parameterOffset = MethodUtil.isStatic(mn) ? 0 : 1; // in certain case, local variable table of non-static method can be empty (issue-136)
int parameterOffset = MethodUtil.isStatic(mn) ? 0 : Math.min(mn.localVariables.size(), 1);
mn.localVariables.add(parameterOffset, new LocalVariableNode(SELF_REF, targetClassName, null, mn.localVariables.add(parameterOffset, new LocalVariableNode(SELF_REF, targetClassName, null,
labels.left, labels.right, parameterOffset)); labels.left, labels.right, parameterOffset));
for (int i = parameterOffset + 1; i < mn.localVariables.size(); i++) { for (int i = parameterOffset + 1; i < mn.localVariables.size(); i++) {
@ -123,6 +124,7 @@ public class MockClassHandler extends BaseClassWithContextHandler {
} }
private ImmutablePair<LabelNode, LabelNode> getStartAndEndLabel(MethodNode mn) { private ImmutablePair<LabelNode, LabelNode> getStartAndEndLabel(MethodNode mn) {
// in certain case, local variable table of non-static method can be empty (issue-136)
if (MethodUtil.isStatic(mn) || mn.localVariables.isEmpty()) { if (MethodUtil.isStatic(mn) || mn.localVariables.isEmpty()) {
LabelNode startLabel = null, endLabel = null; LabelNode startLabel = null, endLabel = null;
for (AbstractInsnNode n = mn.instructions.getFirst(); n != null; n = n.getNext()) { for (AbstractInsnNode n = mn.instructions.getFirst(); n != null; n = n.getNext()) {