Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

London test fixes #111

Merged
merged 16 commits into from
Feb 19, 2024
Prev Previous commit
Next Next commit
Suppress linting warnings
  • Loading branch information
Stefan-Ethernal authored and goran-ethernal committed Feb 16, 2024
commit 24c4fe67a9614d05dfa36d2175baf01b686a2a35
3 changes: 2 additions & 1 deletion state/executor.go
Original file line number Diff line number Diff line change
Expand Up @@ -698,7 +698,7 @@ func (t *Transition) apply(msg *types.Transaction) (*runtime.ExecutionResult, er
coinbaseFee := new(big.Int).Mul(new(big.Int).SetUint64(result.GasUsed), effectiveTip)
t.state.AddBalance(t.ctx.Coinbase, coinbaseFee)

// nolint:godox
//nolint:godox
// TODO - burning of base fee should not be done in the EVM
// Burn some amount if the london hardfork is applied.
// Basically, burn amount is just transferred to the current burn contract.
Expand Down Expand Up @@ -1342,6 +1342,7 @@ func (t *Transition) RevertToSnapshot(snapshot int) error {
})

if idx == len(t.journalRevisions) || t.journalRevisions[idx].ID != snapshot {
//nolint:gocritic
panic(fmt.Errorf("journal revision id %v cannot be reverted", snapshot))
}

Expand Down
Loading