diff options
| author | MarcoFalke <[email protected]> | 2017-11-10 15:33:28 -0500 |
|---|---|---|
| committer | MarcoFalke <[email protected]> | 2017-11-10 15:33:30 -0500 |
| commit | 61fb80660f73e5aa5b69302ecc7ac33da206ba5a (patch) | |
| tree | 1b7b5b18cb3b7f989de87847ddaeb49e24f4fa29 /src/txmempool.cpp | |
| parent | Merge #11353: Small refactor of CCoinsViewCache::BatchWrite() (diff) | |
| parent | Fix CTxMemPoolEntry::UpdateAncestorState: modifySigOps param type int -> int64_t (diff) | |
| download | discoin-61fb80660f73e5aa5b69302ecc7ac33da206ba5a.tar.xz discoin-61fb80660f73e5aa5b69302ecc7ac33da206ba5a.zip | |
Merge #11269: [Mempool] CTxMemPoolEntry::UpdateAncestorState: modifySiagOps param type
203a4aa31 Fix CTxMemPoolEntry::UpdateAncestorState: modifySigOps param type int -> int64_t (donaloconnor)
Pull request description:
CTxMemPoolEntry::CTxMemPoolEntry's modifySigOps parameter is int while update_ancestor_state::modifySigOpsCost is int64_t. This issue was raised in #11165. It looks like the function paramaters were not changed in commit 72abd2c
This will avoid unexpected truncation of int64_t -> int
Tree-SHA512: 314c703f217e104336456859066d18fb0d12c4f9f32835e17490a6f29eb05951184095039e4e57edacef8ad35dd75c6d97d9af656a52209dd0c3779b4ffa0914
Diffstat (limited to 'src/txmempool.cpp')
| -rw-r--r-- | src/txmempool.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/txmempool.cpp b/src/txmempool.cpp index b0306811c..c509e22a2 100644 --- a/src/txmempool.cpp +++ b/src/txmempool.cpp @@ -317,7 +317,7 @@ void CTxMemPoolEntry::UpdateDescendantState(int64_t modifySize, CAmount modifyFe assert(int64_t(nCountWithDescendants) > 0); } -void CTxMemPoolEntry::UpdateAncestorState(int64_t modifySize, CAmount modifyFee, int64_t modifyCount, int modifySigOps) +void CTxMemPoolEntry::UpdateAncestorState(int64_t modifySize, CAmount modifyFee, int64_t modifyCount, int64_t modifySigOps) { nSizeWithAncestors += modifySize; assert(int64_t(nSizeWithAncestors) > 0); |