Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Test kEmptyRoot #631

Merged
merged 2 commits into from
Apr 7, 2022
Merged

Test kEmptyRoot #631

merged 2 commits into from
Apr 7, 2022

Conversation

yperbasis
Copy link
Member

Finally, after ethereum/yellowpaper#852, I understand where the empty root hash comes from.

chfast
chfast approved these changes Apr 7, 2022
View reviewed changes
Copy link
Member

@chfast chfast left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, the Trie defines special null node which is only used to indicate empty trie. It is defined as empty rlp string, but I don't see any specific reason for it.

@yperbasis yperbasis merged commit d52bc6c into master Apr 7, 2022
@yperbasis yperbasis deleted the empty_root branch April 7, 2022 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chfast chfast chfast approved these changes

@AndreaLanfranchi AndreaLanfranchi Awaiting requested review from AndreaLanfranchi

@canepat canepat Awaiting requested review from canepat

@enriavil1 enriavil1 Awaiting requested review from enriavil1

@Giulio2002 Giulio2002 Awaiting requested review from Giulio2002

@mriccobene mriccobene Awaiting requested review from mriccobene

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@yperbasis @chfast