Closed seolaoh closed 2 weeks ago
The changes involve modifications to the zkMerkleTreeNodeBlobFunctions
and zktrieNodeBlobFunctions
functions in the trie/iterator.go
file, specifically altering how empty nodes are handled by returning a canonical value instead of nil. Additionally, new test cases have been added in trie/iterator_test.go
to validate the behavior of the iterators when dealing with empty root nodes in zk merkle trees and zk tries, enhancing test coverage for these edge cases.
File | Change Summary |
---|---|
trie/iterator.go |
Updated function signatures for zkMerkleTreeNodeBlobFunctions and zktrieNodeBlobFunctions . Altered return value for empty nodes in zkMerkleTreeNodeBlobFunctions . |
trie/iterator_test.go |
Added new test cases for empty root nodes in zk merkle trees and zk tries, including TestMerkleTreeIterator , TestEmptyRoot , TestZkMerkleTree , and TestZkTrie . |
sequenceDiagram
participant User
participant Iterator
participant Trie
User->>Iterator: Initialize with empty trie
Iterator->>Trie: Check for nodes
Trie-->>Iterator: No nodes found
Iterator->>User: Return empty result
[!WARNING] There were issues while running some tools. Please review the errors and either fix the tool’s configuration or disable the tool if it’s a critical failure.
🔧 golangci-lint
level=warning msg="[config_reader] The configuration option `run.skip-files` is deprecated, please use `issues.exclude-files`." level=warning msg="The linter 'exportloopref' is deprecated (since v1.60.2) due to: Since Go1.22 (loopvar) this linter is no longer relevant. Replaced by copyloopvar."
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Description
This PR is for release
v0.5.4
.