status-im/nimbus-eth1
2
0
Fork 0
mirror of https://github.com/status-im/nimbus-eth1.git synced 2025-02-03 07:45:18 +00:00
Code Issues Projects Releases Wiki Activity

s/increaseBalance/addBalance

Browse Source
This commit is contained in:
Yuriy Glukhov 2018-09-19 12:37:31 +03:00 committed by zah
parent 75f973e654
commit 0adfaf0f31
3 changed files with 9 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
nimbus/db/state_db.nim
View File

@ -61,9 +61,6 @@ proc setBalance*(db: var AccountStateDB, address: EthAddress, balance: UInt256)
account.balance = balance
db.setAccount(address, account)
proc increaseBalance*(db: var AccountStateDB, address: EthAddress, delta: UInt256) =
db.setBalance(address, db.getBalance(address) + delta)
proc addBalance*(db: var AccountStateDB, address: EthAddress, delta: UInt256) =
db.setBalance(address, db.getBalance(address) + delta)

2
nimbus/vm/computation.nim
View File

@ -105,7 +105,7 @@ proc applyMessage(computation: var BaseComputation) =
computation.vmState.mutateStateDb:
db.setBalance(computation.msg.sender, db.getBalance(computation.msg.sender) - computation.msg.value)
db.increaseBalance(computation.msg.storage_address, computation.msg.value)
db.addBalance(computation.msg.storage_address, computation.msg.value)
debug "Apply message",
value = computation.msg.value,

16
tests/test_generalstate_json.nim
View File

@ -72,7 +72,7 @@ proc testFixture(fixtures: JsonNode, testStatusIMPL: var TestStatus) =
vmState.mutateStateDb:
# pre-EIP158 (e.g., Byzantium, should ensure currentCoinbase exists)
# but in later forks, don't create at all
db.increaseBalance(currentCoinbase, 0.u256)
db.addBalance(currentCoinbase, 0.u256)
# FIXME: don't repeat this code
# TODO: iterate over all fixture indexes
@ -92,7 +92,7 @@ proc testFixture(fixtures: JsonNode, testStatusIMPL: var TestStatus) =
# Also, in general, map out/etc the whole vmState.mutateStateDB flow set
db.setBalance(sender, db.getBalance(sender) - gas_cost)
db.setNonce(sender, db.getNonce(sender) + 1)
db.increaseBalance(transaction.to, transaction.value)
db.addBalance(transaction.to, transaction.value)
db.setBalance(sender, db.getBalance(sender) - transaction.value)
# build_message (Py-EVM)
@ -139,8 +139,8 @@ proc testFixture(fixtures: JsonNode, testStatusIMPL: var TestStatus) =
vmState.mutateStateDB:
if currentCoinbase notin deletedAccounts:
db.setBalance(currentCoinbase, db.getBalance(currentCoinbase) - gasRefundAmount)
db.increaseBalance(currentCoinbase, gas_cost)
db.increaseBalance(sender, gasRefundAmount)
db.addBalance(currentCoinbase, gas_cost)
db.addBalance(sender, gasRefundAmount)
# TODO: only here does one commit, with some nuance/caveat
else:
# XXX: both error paths are intentionally indentical, for merging, with refactoring
@ -148,17 +148,17 @@ proc testFixture(fixtures: JsonNode, testStatusIMPL: var TestStatus) =
vmState.mutateStateDB:
# XXX: the coinbase has to be committed; the rest are basically reverts
db.setBalance(transaction.to, db.getBalance(transaction.to) - transaction.value)
db.increaseBalance(sender, transaction.value)
db.addBalance(sender, transaction.value)
db.setStorageRoot(transaction.to, storageRoot)
db.increaseBalance(currentCoinbase, gas_cost)
db.addBalance(currentCoinbase, gas_cost)
except ValueError:
# TODO: replace with transactional commit/revert state (foo.revert or implicit)
vmState.mutateStateDB:
# XXX: the coinbase has to be committed; the rest are basically reverts
db.setBalance(transaction.to, db.getBalance(transaction.to) - transaction.value)
db.increaseBalance(sender, transaction.value)
db.addBalance(sender, transaction.value)
db.setStorageRoot(transaction.to, storageRoot)
db.increaseBalance(currentCoinbase, gas_cost)
db.addBalance(currentCoinbase, gas_cost)
#echo vmState.readOnlyStateDB.dumpAccount("b94f5374fce5edbc8e2a8697c15331677e6ebf0b")
#echo vmState.readOnlyStateDB.dumpAccount("a94f5374fce5edbc8e2a8697c15331677e6ebf0b")