VM trait/struct refactoring #153

sorpaas · 2017-06-01T09:47:25Z

Part of #152.

Solution: add used_gas fn

Solution: give it a name: machines

Solution: remove VMError, use MachineError directly

Solution: add back transaction enum

Solution: make VM a trait, and change the original VM to ContextVM

coveralls · 2017-06-01T09:59:13Z

Coverage decreased (-0.1%) to 77.935% when pulling 6cf0186 on sorpaas:master into 865d83a on ethereumproject:master.

coveralls · 2017-06-01T10:08:16Z

Coverage decreased (-0.1%) to 77.935% when pulling 6cf0186 on sorpaas:master into 865d83a on ethereumproject:master.

coveralls · 2017-06-01T10:08:16Z

Coverage decreased (-0.1%) to 77.935% when pulling 6cf0186 on sorpaas:master into 865d83a on ethereumproject:master.

coveralls · 2017-06-01T10:08:16Z

Coverage decreased (-0.1%) to 77.935% when pulling 6cf0186 on sorpaas:master into 865d83a on ethereumproject:master.

sorpaas and others added 7 commits June 1, 2017 17:00

Problem: missing used_gas fn in VM

c4cdcb8

Solution: add used_gas fn

Problem: VM's self.0 is not totally clear

aa4ca04

Solution: give it a name: machines

Problem: VMError does not provide additional abstraction to MachineError

bc37b09

Solution: remove VMError, use MachineError directly

Problem: ETCDEVTeam#152 wrapping transactions for VM

38209f8

Solution: add back transaction enum

Problem: ETCDEVTeam#152 wrapping transactions for VM

05dc87d

Solution: make VM a trait, and change the original VM to ContextVM

Merge branch 'master' of github.com:sorpaas/sputnikvm

f54bc7e

Merge branch 'master' into master

6cf0186

sjmackenzie merged commit 62c6667 into ETCDEVTeam:master Jun 1, 2017

Provide feedback