diff options
author | Pieter Wuille <pieter.wuille@gmail.com> | 2017-06-13 12:17:30 -0700 |
---|---|---|
committer | Pieter Wuille <pieter.wuille@gmail.com> | 2017-06-26 16:16:26 -0700 |
commit | 21180ff73436e198b6828c312ddfd0a1195447b2 (patch) | |
tree | 32541dfda7113742cc54340eda7e859ab62869ce /src/txmempool.h | |
parent | 234ffc677ee58591b139695bf92bbd6f504ee91a (diff) |
Simplify return values of GetCoin/HaveCoin(InCache)
This removes the possibility for GetCoin/HaveCoin/HaveCoinInCache to return
true while the respective coin is spent. By doing it across all calls, some
extra checks can be eliminated.
coins_tests is modified to call HaveCoin sometimes before and sometimes
after AccessCoin. A further change is needed because the semantics for
GetCoin slightly changed, causing a pruned entry in the parent cache to not
be pulled into the child in FetchCoin.
Diffstat (limited to 'src/txmempool.h')
-rw-r--r-- | src/txmempool.h | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/txmempool.h b/src/txmempool.h index 7ca3b18a1e..78ac3c209b 100644 --- a/src/txmempool.h +++ b/src/txmempool.h @@ -684,8 +684,7 @@ protected: public: CCoinsViewMemPool(CCoinsView* baseIn, const CTxMemPool& mempoolIn); - bool GetCoin(const COutPoint &outpoint, Coin &coin) const; - bool HaveCoin(const COutPoint &outpoint) const; + bool GetCoin(const COutPoint &outpoint, Coin &coin) const override; }; /** |