Remove unused variable (cacheMap) in mempool
What the developer wrote
Authored by Suhas Daftuar
Remove unused variable (cacheMap) in mempool
What changed, and why it matters
This commit simply removes an unused type alias named cacheMap from the transaction mempool header file. It is a routine code cleanup with no functional change and no security relevance.
No security action needed. Treat as normal maintenance cleanup.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff deletes a single private typedef declaration (typedef std::map<txiter, setEntries, CompareIteratorByHash> cacheMap;) from CTxMemPool in src/txmempool.h. The type is not referenced anywhere in the codebase, so this is a dead-code removal. There are no logic, API, or behavioral changes.
Changed components
src/txmempool.hInspect captured patch +0 / −3
diff --git a/src/txmempool.h b/src/txmempool.h
index 1fc94c6d..c305d5d4 100644
--- a/src/txmempool.h
+++ b/src/txmempool.h
@@ -281,9 +281,6 @@ public:
std::vector<CTxMemPoolEntry::CTxMemPoolEntryRef> GetParents(const CTxMemPoolEntry &entry) const;
private:
- typedef std::map<txiter, setEntries, CompareIteratorByHash> cacheMap;
-
-
std::vector<indexed_transaction_set::const_iterator> GetSortedScoreWithTopology() const EXCLUSIVE_LOCKS_REQUIRED(cs);
/**
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.