remove unnecessary empty lines in PathToLeaf.String() (#349)

cosmos · Dec 14, 2020 · c140037 · c140037
1 parent b3b6e2a
commit c140037
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -6,6 +6,8 @@
 
 - [\#347](https://github.com/cosmos/iavl/pull/347) Fix another integer overflow in `decodeBytes()` that can cause panics for certain inputs. The `ValueOp` and `AbsenceOp` proof decoders are vulnerable to this via malicious inputs since 0.15.0.
 
+- [\#349](https://github.com/cosmos/iavl/pull/349) Fix spurious blank lines in `PathToLeaf.String()`.
+
 ## 0.15.1 (December 13, 2020)
 
 ### Bug Fixes

diff --git a/proof_path.go b/proof_path.go
@@ -47,13 +47,13 @@ func (pl PathToLeaf) stringIndented(indent string) string {
  if len(pl) == 0 {
  return "empty-PathToLeaf"
  }
- strs := make([]string, len(pl))
+ strs := make([]string, 0, len(pl))
  for i, pin := range pl {
  if i == 20 {
- strs[i] = fmt.Sprintf("... (%v total)", len(pl))
+ strs = append(strs, fmt.Sprintf("... (%v total)", len(pl)))
  break
  }
- strs[i] = fmt.Sprintf("%v:%v", i, pin.stringIndented(indent+" "))
+ strs = append(strs, fmt.Sprintf("%v:%v", i, pin.stringIndented(indent+" ")))
  }
  return fmt.Sprintf(`PathToLeaf{
 %s %v