Skip to content

Conversation

@AJStonewee
Copy link
Contributor

Fixed incorrect comment in sparse_trie_reveal_node_1 test: changed "hash builder has nodes 0x00 and 0x01" to "0x00 and 0x02" to match the actual code implementation which uses key1() and key3().

@AJStonewee
Copy link
Contributor Author

// in line 3167:hash builder use key1 (0x00) and key3 (0x02)
[(key1(), value()), (key3(), value())]

// in line 3211:insert new node key2 (0x01)
sparse.update_leaf(key2(), value_encoded(), &provider).unwrap();

so in fact :
Hash builder:0x00 and 0x02
new and inserted node: 0x01

@mattsse mattsse added the C-docs An addition or correction to our documentation label Oct 23, 2025
@github-project-automation github-project-automation bot moved this from Backlog to In Progress in Reth Tracker Oct 24, 2025
@mediocregopher mediocregopher added this pull request to the merge queue Oct 24, 2025
Merged via the queue into paradigmxyz:main with commit f177103 Oct 24, 2025
42 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Reth Tracker Oct 24, 2025
@jenpaff jenpaff moved this from Done to Completed in Reth Tracker Oct 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-docs An addition or correction to our documentation

Projects

Status: Completed

Development

Successfully merging this pull request may close these issues.

3 participants