-
Notifications
You must be signed in to change notification settings - Fork 351
EIP-2929: only add builtin to warm address if they are active #364
base: dev
Are you sure you want to change the base?
Conversation
Oh do tell :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indeed "ActivePrecompiles" only includes warm addresses.
Ref: ethereum/go-ethereum@d990f93
I can confirm I was able to go past the problematic block after applying this patch. |
@sorpaas can you rebase that two commint to release/v3.2.1 branch, so that only that change is included into hot patch? |
Will do! |
LGTM |
Its synchronizing good job @sorpaas waiting to be sync and then we can say that was it |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
LGTM ! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good to me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM :)
@alexprut 大哥。。。就是找你顺手帮个忙。。100XRP酬谢。。不帮也说一声就好了。。。屏蔽我不至于吧··· |
The actual fix of #353. Added here just for everyone to test. Credits go to the bug finder @holiman!