Skip to content

EVM/Kernel: [block::new] and [block::first_block] needs the gas limit

Valentin Chaboche requested to merge vch9@evm-gas-limit-in-new-block into master

What

It should be luckily equivalent. When the block is stored we use
the default value of 3 fields: gas_limit, miner and extraData. As the
default value of gas limit is already 2^50 it works fine, but if you
change default value in the crate ethereum, you'll have a different
gas limit in the opcode and in the block.

Checklist

  • Document the interface of any function added or modified (see the coding guidelines)
  • Document any change to the user interface, including configuration parameters (see node configuration)
  • Provide automatic testing (see the testing guide).
  • For new features and bug fixes, add an item in the appropriate changelog (docs/protocols/alpha.rst for the protocol and the environment, CHANGES.rst at the root of the repository for everything else).
  • Select suitable reviewers using the Reviewers field below.
  • Select as Assignee the next person who should take action on that MR

Merge request reports