Skip to content
This repository has been archived by the owner on Apr 4, 2024. It is now read-only.

chore(evm): add comment regarding RANDOM opcode #1520

Merged
merged 4 commits into from
Dec 2, 2022
Merged

chore(evm): add comment regarding RANDOM opcode #1520

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
9fbe153
fix: add random field to BlockContext using block header hash
austinchandra Nov 30, 2022
ebe1589
Merge branch 'main' of github.com:evmos/ethermint into austinchandra/…
fedekunze Dec 2, 2022
1cefad9
add comment
fedekunze Dec 2, 2022
0c4b67a
Merge branch 'main' into austinchandra/ENG-1512
fedekunze Dec 2, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions x/evm/keeper/state_transition.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -72,6 +72,11 @@ func (k *Keeper) TxConfig(ctx sdk.Context, txHash common.Hash) statedb.TxConfig
// (ChainConfig and module Params). It additionally sets the validator operator address as the
// coinbase address to make it available for the COINBASE opcode, even though there is no
// beneficiary of the coinbase transaction (since we're not mining).
//
// NOTE: the RANDOM opcode is currently not supported since it requires
// RANDAO implementation. See https://github.com/evmos/ethermint/pull/1520#pullrequestreview-1200504697
// for more information.

func (k *Keeper) NewEVM(
ctx sdk.Context,
msg core.Message,
Expand All @@ -89,6 +94,7 @@ func (k *Keeper) NewEVM(
Time: big.NewInt(ctx.BlockHeader().Time.Unix()),
Difficulty: big.NewInt(0), // unused. Only required in PoW context
BaseFee: cfg.BaseFee,
Random: nil, // not supported
}

txCtx := core.NewEVMTxContext(msg)
Expand Down