-
Notifications
You must be signed in to change notification settings - Fork 70
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
Revert "Fixup merkle-value" #702
Conversation
IMO we should maybe just get rid off this merkle value. This is used no where in the spec and I only see it bringing confusion, than any help in understanding the spec or whatever. |
It is referenced here, in the definition of the subvalue of a trie node. |
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.
@tjjfvi and @bkchr, thanks a lot for the PR and the active discussion/contribution here. If you plan to contribute more, I'm also happy to add you to the spec committee. https://github.com/w3f/polkadot-spec#spec-committee (approve PRs).
@Noc2 Thanks; I would definitely be open to joining the committee. (I have been working on implementing the protocol, and do plan to contribute fixes back to the spec when I have time.) |
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.
Definition reverted correctly.
Co-authored-by: Sebastian Kunert <[email protected]>
Reverts #698
As discussed in the comments of that PR, the behavior described before by the spec was correct – a node is inlined only if
len < 32
.