remove redundant hashing on commit

crypto/merkle/proof_simple_value.go

@@ -5,7 +5,6 @@ import (
 	"fmt"
 
 	"github.com/pkg/errors"
-
 	"github.com/tendermint/tendermint/crypto/tmhash"
 )
 

crypto/merkle/simple_map.go

@@ -5,7 +5,6 @@ import (
 
 	amino "github.com/tendermint/go-amino"
 
-	"github.com/tendermint/tendermint/crypto/tmhash"
 	"github.com/tendermint/tendermint/libs/kv"
 )
 
@@ -29,14 +28,9 @@ func newSimpleMap() *simpleMap {
 func (sm *simpleMap) Set(key string, value []byte) {
 	sm.sorted = false
 
-	// The value is hashed, so you can
-	// check for equality with a cached value (say)
-	// and make a determination to fetch or not.
-	vhash := tmhash.Sum(value)
-
 	sm.kvs = append(sm.kvs, kv.Pair{
 		Key:   []byte(key),
-		Value: vhash,
+		Value: value,
 	})
 }
 
@@ -71,6 +65,15 @@ func (sm *simpleMap) KVPairs() kv.Pairs {
 // then hashed.
 type KVPair kv.Pair
 
+// NewKVPair takes in a key and value and creates a kv.Pair
+// wrapped in the local extension KVPair
+func NewKVPair(key, value []byte) KVPair {
+	return KVPair(kv.Pair{
+		Key:   key,
+		Value: value,
+	})
+}
+
 // Bytes returns key || value, with both the
 // key and value length prefixed.
 func (kv KVPair) Bytes() []byte {