Pluggable consensus (develop cache) #647
Commits on Jun 5, 2018
Commits on Jun 11, 2018
Commits on Jun 13, 2018
Commits on Jun 15, 2018
Commits on Jun 21, 2018
-
-
-
problem: Blockchain doesn't implement vm ChainContext
solution: add unused Engine() method
-
problem: vm want Message<->Tx struct+interfaces
solution: implement them
-
problem: state processor doesn't use consensus engine
solution: remove references
-
problem: statedb.Prepare method doesn't exist
solution: use original StartRecord method
-
problem: vm doesn't have vm.Logs
solution: shopt -s globstar && sed -i 's/vm\.Logs/\[\]\*types\.Log/g' ./**/*.go
-
problem: vm doesn't have type Account
solution: sed vm.Account -> vm.AccountRef
Commits on Jun 22, 2018
-
problem: StateDB must implement vm StateDB interface
solution: modify and add methods to fit
-
problem: unimplemented feature/tx-status changes
solution: merge tzdybal's upstream/feature/tx-status changes, resolving some conflicts and creating some conflicts
-
problem: receipt status logic moved to receipt.go
solution: just move logic to NewReceipt fn
-
problem: core/*.go still references local Default configs
solution: sed -> params.
-
problem: unused imports, typos, and general compilation errors
solution: remove unused imports and invalid methods, fix calls to types.Log
-
problem: prefixedHex not exported for GenesisDump cmd
solution: export it * IMO This is really fuckin ugly. TODO, unexport, refactor
-
problem: mismatched signatures and unreferenced vars
solution: fix - use vm.Config in blockchain#StateProcessor signature - change returned gas value to uint64 type for ^ method - reference params.TxGas - ...
-
problem: state object not created, causing blockchain test to panic
solution: ensure dirties are checked in statedb
-
problem: rename state#StartRecord -> Prepare
- also, whitespace and just fn ordering
-
problem: vm.Config is wrong arg type for Process
should be consensus.Engine solution: remove entirely
-
Commits on Jun 23, 2018
-
problem: statedb tests fail - wrong refund values
solution: use uint64 instead of big.Int
-
problem: statedb_test snapshot tests failing
solution: move to uint64 refund and gas values across the board
-
problem: block GasUsed is 0 in tests
solution: pass address of block header GasUsed
-
problem: decoding Receipt RLP fails (too many inputs)
solution: revert merged changes from 'status' field PR
-
problem: nil pointer TestChainConfig for core/ tests
solution: set value during defaults
-
problem: ethdb.NewMemDatabase returns multiple vals in single val con…
…text solution: fix, minor refactor
-
problem: block tests fails because mismatched gas types
solution: use uint64 instead of *big.Int
-
problem: vm/instructions_test.go fails on shift tests with 'ff'
solution: common/big Big256 isn't 256. It's 255. WTF. https://play.golang.org/p/S7UZiIgkKWM
-
-
-
problem: abi/simulated used antiquated signatures and types
solution: update
-
-
problem: cmd/geth tests failing
solution: fix uint64 vs. bigInts and params imports
Commits on Jun 25, 2018
Commits on Jun 27, 2018
-
-
-
-
problem: genesis types incompatible
solution: create adhoc core.Genesis type to fit tests
-
problem: state tests runner incompatible with tests/ pub fns
solution: comment, temporarily
-
problem: genesis and state tests unmarshaling
solution: use generated marshalers
-
problem: state test post root always fails
solution: comment line; TBD
-
problem: core/multivm_processor using old signatures
solution: update gas type, log and receipt signatures
-
problem: makefile command use old config path
solution: move from core/config -> params/config
-
problem: typos in multivm_processor
solution: fix 'em. happen cuz of a sed gone awry
-
Merge remote-tracking branch 'whilei/mini-moaf-statetests' into issue…
…/pluggable_consensus