diff options
| author | glozow <gloriajzhao@gmail.com> | 2022-01-17 10:24:38 +0000 |
|---|---|---|
| committer | glozow <gloriajzhao@gmail.com> | 2022-01-18 11:55:15 +0000 |
| commit | e177fcab3831b6d259da5164cabedcc9e78f6957 (patch) | |
| tree | b3740b0ff6c23721dd6b9fba3627e90a323ff7e1 /src | |
| parent | c7cd98c7176800a51e6a6b3634a26b508aa33ff2 (diff) | |
Replace `struct update_lock_points` with lambda
No behavior change.
This code was introduced in 5add7a7 before we required C++11, which is
why the struct was needed. As we are now using more modern C++ and this
is the only place where lockpoints are updated for mempool entries, it
is more idiomatic to call `modify` with a lambda.
Co-authored-by: Hennadii Stepanov <32963518+hebasto@users.noreply.github.com>
Diffstat (limited to 'src')
| -rw-r--r-- | src/txmempool.h | 10 | ||||
| -rw-r--r-- | src/validation.cpp | 2 |
2 files changed, 1 insertions, 11 deletions
diff --git a/src/txmempool.h b/src/txmempool.h index 9fc3054e3c..e025dafd91 100644 --- a/src/txmempool.h +++ b/src/txmempool.h @@ -312,16 +312,6 @@ public: } }; -struct update_lock_points -{ - explicit update_lock_points(const LockPoints& _lp) : lp(_lp) { } - - void operator() (CTxMemPoolEntry &e) { e.UpdateLockPoints(lp); } - -private: - const LockPoints& lp; -}; - // Multi_index tag names struct descendant_score {}; struct entry_time {}; diff --git a/src/validation.cpp b/src/validation.cpp index b2d253afd2..fff7cfc07b 100644 --- a/src/validation.cpp +++ b/src/validation.cpp @@ -375,7 +375,7 @@ void CChainState::MaybeUpdateMempoolForReorg( } else if (!validLP) { // If CheckSequenceLocks succeeded, it also updated the LockPoints. // Now update the mempool entry lockpoints as well. - m_mempool->mapTx.modify(it, update_lock_points(lp)); + m_mempool->mapTx.modify(it, [&lp](CTxMemPoolEntry& e) { e.UpdateLockPoints(lp); }); } // If the transaction spends any coinbase outputs, it must be mature. |