From bbc4ea4ae8e8a962deae3d5693d9d4a9376eab88 Mon Sep 17 00:00:00 2001 From: Jeffrey Wilcke Date: Thu, 5 Jan 2017 11:52:10 +0100 Subject: core/vm: improved EVM run loop & instruction calling (#3378) The run loop, which previously contained custom opcode executes have been removed and has been simplified to a few checks. Each operation consists of 4 elements: execution function, gas cost function, stack validation function and memory size function. The execution function implements the operation's runtime behaviour, the gas cost function implements the operation gas costs function and greatly depends on the memory and stack, the stack validation function validates the stack and makes sure that enough items can be popped off and pushed on and the memory size function calculates the memory required for the operation and returns it. This commit also allows the EVM to go unmetered. This is helpful for offline operations such as contract calls. --- miner/worker.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'miner') diff --git a/miner/worker.go b/miner/worker.go index f29566c0a..3ae51d120 100644 --- a/miner/worker.go +++ b/miner/worker.go @@ -616,7 +616,7 @@ func (env *Work) commitTransactions(mux *event.TypeMux, txs *types.TransactionsB func (env *Work) commitTransaction(tx *types.Transaction, bc *core.BlockChain, gp *core.GasPool) (error, vm.Logs) { snap := env.state.Snapshot() - receipt, logs, _, err := core.ApplyTransaction(env.config, bc, gp, env.state, env.header, tx, env.header.GasUsed, vm.Config{}) + receipt, _, err := core.ApplyTransaction(env.config, bc, gp, env.state, env.header, tx, env.header.GasUsed, vm.Config{}) if err != nil { env.state.RevertToSnapshot(snap) return err, nil @@ -624,7 +624,7 @@ func (env *Work) commitTransaction(tx *types.Transaction, bc *core.BlockChain, g env.txs = append(env.txs, tx) env.receipts = append(env.receipts, receipt) - return nil, logs + return nil, receipt.Logs } // TODO: remove or use -- cgit v1.2.3