forks/go-ethereum
1
0
Fork 1
mirror of https://github.com/ethereum/go-ethereum.git synced 2026-08-19 10:22:23 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

core/state: drop Journal-prefix on journal methods

Browse source
This commit is contained in:
Martin Holst Swende 2024-08-21 09:18:36 +02:00
parent 2a2c432410
commit d6ee4d95d2
No known key found for this signature in database
GPG key ID: 683B438C05A5DDF0
5 changed files with 28 additions and 28 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

24
core/state/journal.go
View file

@ -148,30 +148,30 @@ func (j *journal) copy() *journal {
} }
} }
func (j *journal) JournalAccessListAddAccount(addr common.Address) { func (j *journal) AccessListAddAccount(addr common.Address) {
j.append(accessListAddAccountChange{&addr}) j.append(accessListAddAccountChange{&addr})
} }
func (j *journal) JournalAccessListAddSlot(addr common.Address, slot common.Hash) { func (j *journal) AccessListAddSlot(addr common.Address, slot common.Hash) {
j.append(accessListAddSlotChange{ j.append(accessListAddSlotChange{
address: &addr, address: &addr,
slot: &slot, slot: &slot,
}) })
} }
func (j *journal) JournalLog(txHash common.Hash) { func (j *journal) Log(txHash common.Hash) {
j.append(addLogChange{txhash: txHash}) j.append(addLogChange{txhash: txHash})
} }
func (j *journal) JournalCreate(addr common.Address) { func (j *journal) Create(addr common.Address) {
j.append(createObjectChange{account: &addr}) j.append(createObjectChange{account: &addr})
} }
func (j *journal) JournalDestruct(addr common.Address) { func (j *journal) Destruct(addr common.Address) {
j.append(selfDestructChange{account: &addr}) j.append(selfDestructChange{account: &addr})
} }
func (j *journal) JournalSetState(addr common.Address, key, prev, origin common.Hash) { func (j *journal) SetStorage(addr common.Address, key, prev, origin common.Hash) {
j.append(storageChange{ j.append(storageChange{
account: &addr, account: &addr,
key: key, key: key,
@ -180,7 +180,7 @@ func (j *journal) JournalSetState(addr common.Address, key, prev, origin common.
}) })
} }
func (j *journal) JournalSetTransientState(addr common.Address, key, prev common.Hash) { func (j *journal) SetTransientState(addr common.Address, key, prev common.Hash) {
j.append(transientStorageChange{ j.append(transientStorageChange{
account: &addr, account: &addr,
key: key, key: key,
@ -188,29 +188,29 @@ func (j *journal) JournalSetTransientState(addr common.Address, key, prev common
}) })
} }
func (j *journal) JournalRefundChange(previous uint64) { func (j *journal) RefundChange(previous uint64) {
j.append(refundChange{prev: previous}) j.append(refundChange{prev: previous})
} }
func (j *journal) JournalBalanceChange(addr common.Address, previous *uint256.Int) { func (j *journal) BalanceChange(addr common.Address, previous *uint256.Int) {
j.append(balanceChange{ j.append(balanceChange{
account: &addr, account: &addr,
prev: previous.Clone(), prev: previous.Clone(),
}) })
} }
func (j *journal) JournalSetCode(address common.Address) { func (j *journal) SetCode(address common.Address) {
j.append(codeChange{account: &address}) j.append(codeChange{account: &address})
} }
func (j *journal) JournalNonceChange(address common.Address, prev uint64) { func (j *journal) NonceChange(address common.Address, prev uint64) {
j.append(nonceChange{ j.append(nonceChange{
account: &address, account: &address,
prev: prev, prev: prev,
}) })
} }
func (j *journal) JournalTouch(address common.Address) { func (j *journal) Touch(address common.Address) {
j.append(touchChange{ j.append(touchChange{
account: &address, account: &address,
}) })

10
core/state/state_object.go
View file

@ -114,7 +114,7 @@ func (s *stateObject) markSelfdestructed() {
} }
func (s *stateObject) touch() { func (s *stateObject) touch() {
s.db.journal.JournalTouch(s.address) s.db.journal.Touch(s.address)
} }
// getTrie returns the associated storage trie. The trie will be opened if it's // getTrie returns the associated storage trie. The trie will be opened if it's
@ -244,7 +244,7 @@ func (s *stateObject) SetState(key, value common.Hash) {
return return
} }
// New value is different, update and journal the change // New value is different, update and journal the change
s.db.journal.JournalSetState(s.address, key, prev, origin) s.db.journal.SetStorage(s.address, key, prev, origin)
s.setState(key, value, origin) s.setState(key, value, origin)
if s.db.logger != nil && s.db.logger.OnStorageChange != nil { if s.db.logger != nil && s.db.logger.OnStorageChange != nil {
s.db.logger.OnStorageChange(s.address, key, prev, value) s.db.logger.OnStorageChange(s.address, key, prev, value)
@ -498,7 +498,7 @@ func (s *stateObject) SubBalance(amount *uint256.Int, reason tracing.BalanceChan
} }
func (s *stateObject) SetBalance(amount *uint256.Int, reason tracing.BalanceChangeReason) { func (s *stateObject) SetBalance(amount *uint256.Int, reason tracing.BalanceChangeReason) {
s.db.journal.JournalBalanceChange(s.address, s.data.Balance) s.db.journal.BalanceChange(s.address, s.data.Balance)
if s.db.logger != nil && s.db.logger.OnBalanceChange != nil { if s.db.logger != nil && s.db.logger.OnBalanceChange != nil {
s.db.logger.OnBalanceChange(s.address, s.Balance().ToBig(), amount.ToBig(), reason) s.db.logger.OnBalanceChange(s.address, s.Balance().ToBig(), amount.ToBig(), reason)
} }
@ -574,7 +574,7 @@ func (s *stateObject) CodeSize() int {
} }
func (s *stateObject) SetCode(codeHash common.Hash, code []byte) { func (s *stateObject) SetCode(codeHash common.Hash, code []byte) {
s.db.journal.JournalSetCode(s.address) s.db.journal.SetCode(s.address)
if s.db.logger != nil && s.db.logger.OnCodeChange != nil { if s.db.logger != nil && s.db.logger.OnCodeChange != nil {
// TODO remove prevcode from this callback // TODO remove prevcode from this callback
s.db.logger.OnCodeChange(s.address, common.BytesToHash(s.CodeHash()), nil, codeHash, code) s.db.logger.OnCodeChange(s.address, common.BytesToHash(s.CodeHash()), nil, codeHash, code)
@ -589,7 +589,7 @@ func (s *stateObject) setCode(codeHash common.Hash, code []byte) {
} }
func (s *stateObject) SetNonce(nonce uint64) { func (s *stateObject) SetNonce(nonce uint64) {
s.db.journal.JournalNonceChange(s.address, s.data.Nonce) s.db.journal.NonceChange(s.address, s.data.Nonce)
if s.db.logger != nil && s.db.logger.OnNonceChange != nil { if s.db.logger != nil && s.db.logger.OnNonceChange != nil {
s.db.logger.OnNonceChange(s.address, s.data.Nonce, nonce) s.db.logger.OnNonceChange(s.address, s.data.Nonce, nonce)
} }

18
core/state/statedb.go
View file

@ -248,7 +248,7 @@ func (s *StateDB) Error() error {
} }
func (s *StateDB) AddLog(log *types.Log) { func (s *StateDB) AddLog(log *types.Log) {
s.journal.JournalLog(s.thash) s.journal.Log(s.thash)
log.TxHash = s.thash log.TxHash = s.thash
log.TxIndex = uint(s.txIndex) log.TxIndex = uint(s.txIndex)
@ -293,14 +293,14 @@ func (s *StateDB) Preimages() map[common.Hash][]byte {
// AddRefund adds gas to the refund counter // AddRefund adds gas to the refund counter
func (s *StateDB) AddRefund(gas uint64) { func (s *StateDB) AddRefund(gas uint64) {
s.journal.JournalRefundChange(s.refund) s.journal.RefundChange(s.refund)
s.refund += gas s.refund += gas
} }
// SubRefund removes gas from the refund counter. // SubRefund removes gas from the refund counter.
// This method will panic if the refund counter goes below zero // This method will panic if the refund counter goes below zero
func (s *StateDB) SubRefund(gas uint64) { func (s *StateDB) SubRefund(gas uint64) {
s.journal.JournalRefundChange(s.refund) s.journal.RefundChange(s.refund)
if gas > s.refund { if gas > s.refund {
panic(fmt.Sprintf("Refund counter below zero (gas: %d > refund: %d)", gas, s.refund)) panic(fmt.Sprintf("Refund counter below zero (gas: %d > refund: %d)", gas, s.refund))
} }
@ -508,7 +508,7 @@ func (s *StateDB) SelfDestruct(addr common.Address) {
// If it is already marked as self-destructed, we do not need to add it // If it is already marked as self-destructed, we do not need to add it
// for journalling a second time. // for journalling a second time.
if !stateObject.selfDestructed { if !stateObject.selfDestructed {
s.journal.JournalDestruct(addr) s.journal.Destruct(addr)
stateObject.markSelfdestructed() stateObject.markSelfdestructed()
} }
} }
@ -531,7 +531,7 @@ func (s *StateDB) SetTransientState(addr common.Address, key, value common.Hash)
if prev == value { if prev == value {
return return
} }
s.journal.JournalSetTransientState(addr, key, prev) s.journal.SetTransientState(addr, key, prev)
s.setTransientState(addr, key, value) s.setTransientState(addr, key, value)
} }
@ -650,7 +650,7 @@ func (s *StateDB) getOrNewStateObject(addr common.Address) *stateObject {
// existing account with the given address, otherwise it will be silently overwritten. // existing account with the given address, otherwise it will be silently overwritten.
func (s *StateDB) createObject(addr common.Address) *stateObject { func (s *StateDB) createObject(addr common.Address) *stateObject {
obj := newObject(s, addr, nil) obj := newObject(s, addr, nil)
s.journal.JournalCreate(addr) s.journal.Create(addr)
s.setStateObject(obj) s.setStateObject(obj)
return obj return obj
} }
@ -1387,7 +1387,7 @@ func (s *StateDB) Prepare(rules params.Rules, sender, coinbase common.Address, d
// AddAddressToAccessList adds the given address to the access list // AddAddressToAccessList adds the given address to the access list
func (s *StateDB) AddAddressToAccessList(addr common.Address) { func (s *StateDB) AddAddressToAccessList(addr common.Address) {
if s.accessList.AddAddress(addr) { if s.accessList.AddAddress(addr) {
s.journal.JournalAccessListAddAccount(addr) s.journal.AccessListAddAccount(addr)
} }
} }
@ -1399,10 +1399,10 @@ func (s *StateDB) AddSlotToAccessList(addr common.Address, slot common.Hash) {
// scope of 'address' without having the 'address' become already added // scope of 'address' without having the 'address' become already added
// to the access list (via call-variant, create, etc). // to the access list (via call-variant, create, etc).
// Better safe than sorry, though // Better safe than sorry, though
s.journal.JournalAccessListAddAccount(addr) s.journal.AccessListAddAccount(addr)
} }
if slotMod { if slotMod {
s.journal.JournalAccessListAddSlot(addr, slot) s.journal.AccessListAddSlot(addr, slot)
} }
} }

2
core/state/statedb_fuzz_test.go
View file

@ -73,7 +73,7 @@ func newStateTestAction(addr common.Address, r *rand.Rand, index int) testAction
args: make([]int64, 1), args: make([]int64, 1),
}, },
{ {
name: "SetState", name: "SetStorage",
fn: func(a testAction, s *StateDB) { fn: func(a testAction, s *StateDB) {
var key, val common.Hash var key, val common.Hash
binary.BigEndian.PutUint16(key[:], uint16(a.args[0])) binary.BigEndian.PutUint16(key[:], uint16(a.args[0]))

2
core/state/statedb_test.go
View file

@ -360,7 +360,7 @@ func newTestAction(addr common.Address, r *rand.Rand) testAction {
args: make([]int64, 1), args: make([]int64, 1),
}, },
{ {
name: "SetState", name: "SetStorage",
fn: func(a testAction, s *StateDB) { fn: func(a testAction, s *StateDB) {
var key, val common.Hash var key, val common.Hash
binary.BigEndian.PutUint16(key[:], uint16(a.args[0])) binary.BigEndian.PutUint16(key[:], uint16(a.args[0]))