-
Notifications
You must be signed in to change notification settings - Fork 4.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
doc: fix Stack size typo in Understanding the Yellow Paper EVM's spec… #10984
Conversation
…ification I read in https://www.evm.codes/about#stack that the Stack has a size of 1024 elements, each of 256 bits, rather than the Stack being of size 1024 bytes (32 elements of 32 bytes)
✅ ethereum-org-website-dev deploy preview ready
|
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.
I feel bad I made the mistake - thank you for fixing it for 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.
Looks good! So sorry for delays here, thank you for patching @Eikix!
Congrats, your important contribution to this open-source project has earned you a GitPOAP! Be sure to join the Ethereum.org discord if you are interested in contributing further to the project or have any questions for the team. GitPOAP: 2024 Ethereum.org Contributor: Head to gitpoap.io & connect your GitHub account to mint! Learn more about GitPOAPs here. |
@all-contributors please add @Eikix for bug fix |
I've put up a pull request to add @Eikix! 🎉 |
✅ Deploy Preview for ethereumorg ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
I read in https://www.evm.codes/about#stack that the Stack has a size of 1024 elements, each of 256 bits, rather than the Stack being of size 1024 bytes (32 elements of 32 bytes)
Description
Related Issue
Resolves #10983