iotex-core icon indicating copy to clipboard operation
iotex-core copied to clipboard

[trie] Fix duplicate flush and remove flush argument

Open Liuhaai opened this issue 3 years ago • 1 comments

store() reflush the children by cn.hash(true)

Liuhaai avatar Mar 14 '22 17:03 Liuhaai

Codecov Report

Merging #3192 (6543c1b) into master (6da40b6) will increase coverage by 0.01%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3192      +/-   ##
==========================================
+ Coverage   75.13%   75.15%   +0.01%     
==========================================
  Files         221      221              
  Lines       21019    21023       +4     
==========================================
+ Hits        15793    15799       +6     
+ Misses       4391     4389       -2     
  Partials      835      835              
Impacted Files Coverage Δ
db/trie/mptrie/node.go 100.00% <ø> (ø)
db/trie/mptrie/branchnode.go 98.58% <100.00%> (-0.03%) :arrow_down:
db/trie/mptrie/cachenode.go 100.00% <100.00%> (ø)
db/trie/mptrie/extensionnode.go 92.98% <100.00%> (ø)
db/trie/mptrie/leafnode.go 100.00% <100.00%> (ø)
db/trie/mptrie/hashnode.go 100.00% <0.00%> (+9.09%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 6da40b6...6543c1b. Read the comment docs.

codecov[bot] avatar Mar 14 '22 17:03 codecov[bot]

Close b/c it has been inactive for 1+yr.

raullenchai avatar May 30 '24 20:05 raullenchai