remove extra call to Exists

Refs #345

https://github.com/tendermint/tendermint/issues/1539#issuecomment-387240606
This commit is contained in:
Anton Kaliaev 2018-05-17 11:52:07 +04:00
parent 0908e668bd
commit ae572b9038
No known key found for this signature in database
GPG Key ID: 7B6881D965918214
1 changed files with 1 additions and 10 deletions

View File

@ -203,10 +203,9 @@ func (mem *Mempool) CheckTx(tx types.Tx, cb func(*abci.Response)) (err error) {
defer mem.proxyMtx.Unlock()
// CACHE
if mem.cache.Exists(tx) {
if !mem.cache.Push(tx) {
return ErrTxInCache
}
mem.cache.Push(tx)
// END CACHE
// WAL
@ -463,14 +462,6 @@ func (cache *txCache) Reset() {
cache.mtx.Unlock()
}
// Exists returns true if the given tx is cached.
func (cache *txCache) Exists(tx types.Tx) bool {
cache.mtx.Lock()
_, exists := cache.map_[string(tx)]
cache.mtx.Unlock()
return exists
}
// Push adds the given tx to the txCache. It returns false if tx is already in the cache.
func (cache *txCache) Push(tx types.Tx) bool {
cache.mtx.Lock()