blockstorage, refactor: make GetFirstStoredBlock() a member of BlockManager

jonatack · jonatack · commit ed12c0a49d3c · 2022-04-28T20:42:08.000+02:00
instead of a global
diff --git a/src/index/base.cpp b/src/index/base.cpp
@@ -75,7 +75,7 @@ bool BaseIndex::Init()
         if (!m_best_block_index) {
             // index is not built yet
             // make sure we have all block data back to the genesis
-            prune_violation = node::GetFirstStoredBlock(active_chain.Tip()) != active_chain.Genesis();
+            prune_violation = m_chainstate->m_blockman.GetFirstStoredBlock(active_chain.Tip()) != active_chain.Genesis();
         }
         // in case the index has a best block set and is not fully synced
         // check if we have the required blocks to continue building the index
diff --git a/src/node/blockstorage.cpp b/src/node/blockstorage.cpp
@@ -390,7 +390,8 @@ bool BlockManager::IsBlockPruned(const CBlockIndex* pblockindex)
     return (m_have_pruned && !(pblockindex->nStatus & BLOCK_HAVE_DATA) && pblockindex->nTx > 0);
 }
 
-const CBlockIndex* GetFirstStoredBlock(const CBlockIndex* start_block) {
+const CBlockIndex* BlockManager::GetFirstStoredBlock(const CBlockIndex* start_block)
+{
     AssertLockHeld(::cs_main);
     assert(start_block);
     const CBlockIndex* last_block = start_block;
diff --git a/src/node/blockstorage.h b/src/node/blockstorage.h
@@ -178,6 +178,9 @@ class BlockManager
     //! Returns last CBlockIndex* that is a checkpoint
     const CBlockIndex* GetLastCheckpoint(const CCheckpointData& data) EXCLUSIVE_LOCKS_REQUIRED(cs_main);
 
+    //! Find the first block that is not pruned
+    const CBlockIndex* GetFirstStoredBlock(const CBlockIndex* start_block) EXCLUSIVE_LOCKS_REQUIRED(::cs_main);
+
     /** True if any block files have ever been pruned. */
     bool m_have_pruned = false;
 
@@ -188,9 +191,6 @@ class BlockManager
     void UpdatePruneLock(const std::string& name, const PruneLockInfo& lock_info) EXCLUSIVE_LOCKS_REQUIRED(::cs_main);
 };
 
-//! Find the first block that is not pruned
-const CBlockIndex* GetFirstStoredBlock(const CBlockIndex* start_block) EXCLUSIVE_LOCKS_REQUIRED(::cs_main);
-
 void CleanupBlockRevFiles();
 
 /** Open a block file (blk?????.dat) */
diff --git a/src/rpc/blockchain.cpp b/src/rpc/blockchain.cpp
@@ -787,7 +787,7 @@ static RPCHelpMan pruneblockchain()
 
     PruneBlockFilesManual(active_chainstate, height);
     const CBlockIndex* block = CHECK_NONFATAL(active_chain.Tip());
-    const CBlockIndex* last_block = node::GetFirstStoredBlock(block);
+    const CBlockIndex* last_block{active_chainstate.m_blockman.GetFirstStoredBlock(block)};
 
     return static_cast<uint64_t>(last_block->nHeight);
 },
@@ -1215,7 +1215,7 @@ RPCHelpMan getblockchaininfo()
     obj.pushKV("size_on_disk", chainman.m_blockman.CalculateCurrentUsage());
     obj.pushKV("pruned", node::fPruneMode);
     if (node::fPruneMode) {
-        obj.pushKV("pruneheight", node::GetFirstStoredBlock(&tip)->nHeight);
+        obj.pushKV("pruneheight", chainman.m_blockman.GetFirstStoredBlock(&tip)->nHeight);
 
         // if 0, execution bypasses the whole if block.
         bool automatic_pruning{args.GetIntArg("-prune", 0) != 1};

Original file line number	Diff line number	Diff line change
`@@ -75,7 +75,7 @@ bool BaseIndex::Init()`
`75`	`75`	`if (!m_best_block_index) {`
`76`	`76`	`// index is not built yet`
`77`	`77`	`// make sure we have all block data back to the genesis`
`78`		`- prune_violation = node::GetFirstStoredBlock(active_chain.Tip()) != active_chain.Genesis();`
	`78`	`+ prune_violation = m_chainstate->m_blockman.GetFirstStoredBlock(active_chain.Tip()) != active_chain.Genesis();`
`79`	`79`	`}`
`80`	`80`	`// in case the index has a best block set and is not fully synced`
`81`	`81`	`// check if we have the required blocks to continue building the index`
Original file line number	Diff line number	Diff line change
`@@ -390,7 +390,8 @@ bool BlockManager::IsBlockPruned(const CBlockIndex* pblockindex)`
`390`	`390`	`return (m_have_pruned && !(pblockindex->nStatus & BLOCK_HAVE_DATA) && pblockindex->nTx > 0);`
`391`	`391`	`}`
`392`	`392`
`393`		`-const CBlockIndex* GetFirstStoredBlock(const CBlockIndex* start_block) {`
	`393`	`+const CBlockIndex* BlockManager::GetFirstStoredBlock(const CBlockIndex* start_block)`
	`394`	`+{`
`394`	`395`	`AssertLockHeld(::cs_main);`
`395`	`396`	`assert(start_block);`
`396`	`397`	`const CBlockIndex* last_block = start_block;`