Codecov Report

Merging #213 into develop will decrease coverage by 0.07%.
The diff coverage is 14.28%.

@@             Coverage Diff             @@
##           develop     #213      +/-   ##
===========================================
- Coverage    58.99%   58.91%   -0.08%     
===========================================
  Files           59       59              
  Lines         3687     3690       +3     
===========================================
- Hits          2175     2174       -1     
- Misses        1360     1364       +4     
  Partials       152      152

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 c3b72ea...e055e88. Read the comment docs.

heads up I'm moving this stuff to go-crypto: tendermint/go-crypto#105

Btw @mossid what prompted this ? Is there some reason the keys needed to be hashed in the first place ?

I had to cache SimpleProofs of subtrees of RootMultiStore. With hashing the key, I had to traverse every possible combination to construct the map.

The comment says that it is for blinding the key, but leaf nodes are hashed anyway so paths don't expose keys. @ebuchman

Moved to tendermint/go-crypto#115