From e9ee78a48dc5edea1af38b8ad874a33b574f4751 Mon Sep 17 00:00:00 2001 From: Adam Date: Sun, 3 Apr 2016 12:40:01 -0400 Subject: [PATCH] Method context was actually important because its copied between frames --- src/main/java/net/runelite/asm/execution/Frame.java | 3 --- src/main/java/net/runelite/asm/execution/MethodContext.java | 2 -- 2 files changed, 5 deletions(-) diff --git a/src/main/java/net/runelite/asm/execution/Frame.java b/src/main/java/net/runelite/asm/execution/Frame.java index 1299f5aa74..602f7b17b5 100644 --- a/src/main/java/net/runelite/asm/execution/Frame.java +++ b/src/main/java/net/runelite/asm/execution/Frame.java @@ -37,9 +37,6 @@ public class Frame stack = new Stack(code.getMaxStack()); variables = new Variables(code.getMaxLocals()); - // don't cache method contexts per execution - // need to allow the same method to execute multiple times - // when called from multiple places to allow graph building //XXX there no longer is a graph ctx = new MethodContext(execution); nonStatic = method; } diff --git a/src/main/java/net/runelite/asm/execution/MethodContext.java b/src/main/java/net/runelite/asm/execution/MethodContext.java index 1559d56fc3..07eda62172 100644 --- a/src/main/java/net/runelite/asm/execution/MethodContext.java +++ b/src/main/java/net/runelite/asm/execution/MethodContext.java @@ -1,8 +1,6 @@ package net.runelite.asm.execution; -import java.util.ArrayList; import java.util.Collection; -import java.util.List; import net.runelite.asm.attributes.code.Instruction; import org.apache.commons.collections4.map.MultiValueMap;