core/vm,params/vars: "correctly" implement EIP2200 SLOAD gas cost change

etclabscore / multi-geth-fork

GNU Lesser General Public License v3.0

3 stars 8 forks source link

core/vm,params/vars: "correctly" implement EIP2200 SLOAD gas cost change #144

Closed meowsbits closed 4 years ago

meowsbits commented 4 years ago

EIP2200 is implemented simultaneously with 1884 on the ETH nets, rendering this somewhat redundant. But other networks may care, and this changeset should be handled carefully, since either or both 2200 1884 may be live with or without the other.

Signed-off-by: meows b5c6@protonmail.com

soc1c commented 4 years ago

restarted travis

soc1c commented 4 years ago

all green: https://travis-ci.org/etclabscore/multi-geth/builds/649849608?utm_source=github_status&utm_medium=notification

meowsbits commented 4 years ago

Releasing this merged version to the testnets, some of which have already "begun" running EIP2200, posing a consistency problem.

It could break the testnets, or?

soc1c commented 4 years ago

Testnets are already broken and need to be rolled back. All we need is releases for Multi Geth and Parity.