-
Notifications
You must be signed in to change notification settings - Fork 837
Open
Labels
Description
In evm there are currently two separate ways to execute bytecode. runCall and runCode. runCode is a legacy of past days when the vm package was more tightly bound to evm. This code is obsolete and unused other than in examples and tests. We should remove it entirely from the code base.
The right solution would do the following:
- Remove the code itself and the associated types from
evm - Remove any examples that reference
runCode - Remove any tests that only test
runCodefunctionality - Revise other tests like here that use
runCodeto instead userunCall. Likely the only difference will be ensuring the returned value fromrunCallaccesses theresult.execResultinstead of justresult.