Skip to content
This repository has been archived by the owner on Jun 17, 2021. It is now read-only.

isPrecompiled() needs to be updated to reflect byzantium #110

Closed
axic opened this issue Jan 5, 2018 · 2 comments
Closed

isPrecompiled() needs to be updated to reflect byzantium #110

axic opened this issue Jan 5, 2018 · 2 comments

Comments

@axic
Copy link
Member

axic commented Jan 5, 2018

See https://github.com/ethereumjs/ethereumjs-util/blob/master/index.js#L519, which limits it to address 0 to 5.

(This feature is used by the -vm and -accounts.)

@axic
Copy link
Member Author

axic commented Jan 5, 2018

Update:

This feature could be removed in the next breaking change, but for now it still needs to be updated.

@holgerd77
Copy link
Member

Yes, removed this from the VM when working on Byzantium. Found this too implicit with too few benefits.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants