Skip to content

fix: encoding of internal nodes in a MPT#16048

Merged
wackerow merged 1 commit into
ethereum:devfrom
trocher:patch-3
Sep 11, 2025
Merged

fix: encoding of internal nodes in a MPT#16048
wackerow merged 1 commit into
ethereum:devfrom
trocher:patch-3

Conversation

@trocher
Copy link
Copy Markdown
Contributor

@trocher trocher commented Aug 11, 2025

Fix documentation of internal node referencing in merkle patricia trie, according to issue #16047

Description

Changed the paragraph to match the actual behavior, according to the execution specs

Related Issue

@netlify
Copy link
Copy Markdown

netlify Bot commented Aug 11, 2025

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit 4186af4
🔍 Latest deploy log https://app.netlify.com/projects/ethereumorg/deploys/6899d83b25459500081d1db2
😎 Deploy Preview https://deploy-preview-16048--ethereumorg.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
7 paths audited
Performance: 54 (no change from production)
Accessibility: 94 (🔴 down 1 from production)
Best Practices: 92 (no change from production)
SEO: 99 (no change from production)
PWA: 59 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions Bot added the content 🖋️ This involves copy additions or edits label Aug 11, 2025
@trocher trocher changed the title fix: encoding of internal nodes in an mpt fix: encoding of internal nodes in a MPT Aug 11, 2025
@github-actions
Copy link
Copy Markdown
Contributor

This issue is stale because it has been open 30 days with no activity.

@github-actions github-actions Bot added the Status: Stale This issue is stale because it has been open 30 days with no activity. label Sep 11, 2025
@wackerow wackerow removed the Status: Stale This issue is stale because it has been open 30 days with no activity. label Sep 11, 2025
Copy link
Copy Markdown
Member

@wackerow wackerow left a comment

Choose a reason for hiding this comment

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

Good catch, looks good.. thanks @trocher for the reference and patching!

@wackerow wackerow merged commit 4050faf into ethereum:dev Sep 11, 2025
6 checks passed
@gitpoap-bot
Copy link
Copy Markdown

gitpoap-bot Bot commented Sep 11, 2025

Congrats, your important contribution to this open-source project has earned you a GitPOAP!

GitPOAP: 2025 Ethereum.org Contributor:

GitPOAP: 2025 Ethereum.org Contributor GitPOAP Badge

Join the [ethereum.org Discord server](https://ethereum.org/discord) to explore more ways to contribute to the project. Depending on the tasks you complete, you may also unlock additional rewards. Visit [ethereum.org/contributing](https://ethereum.org/contributing) to learn more.

Head to gitpoap.io & connect your GitHub account to mint!Keep buidling, keep learning, and let's grow the Ethereum open-source community together 🌱

Learn more about GitPOAPs here.

@wackerow
Copy link
Copy Markdown
Member

@all-contributors please add @trocher for content patch

@allcontributors
Copy link
Copy Markdown
Contributor

@wackerow

I've put up a pull request to add @trocher! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

content 🖋️ This involves copy additions or edits

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Incorrect merkle patricia trie internal node documentation

2 participants