From fa7efc915b87ec56ca1cc0bad7d8f79591bfa099 Mon Sep 17 00:00:00 2001 From: MarcoFalke Date: Sun, 2 Jan 2022 16:59:07 +0100 Subject: [PATCH] Fixup style of moved code Can be reviewed with --word-diff-regex=. -U0 --ignore-all-space --- src/node/blockstorage.cpp | 41 ++++++++++++++++++++------------------- src/node/blockstorage.h | 8 ++++---- 2 files changed, 25 insertions(+), 24 deletions(-) diff --git a/src/node/blockstorage.cpp b/src/node/blockstorage.cpp index 96899fe5a77..60e874967fa 100644 --- a/src/node/blockstorage.cpp +++ b/src/node/blockstorage.cpp @@ -62,8 +62,9 @@ CBlockIndex* BlockManager::AddToBlockIndex(const CBlockHeader& block) // Check for duplicate uint256 hash = block.GetHash(); BlockMap::iterator it = m_block_index.find(hash); - if (it != m_block_index.end()) + if (it != m_block_index.end()) { return it->second; + } // Construct new block index object CBlockIndex* pindexNew = new CBlockIndex(block); @@ -74,8 +75,7 @@ CBlockIndex* BlockManager::AddToBlockIndex(const CBlockHeader& block) BlockMap::iterator mi = m_block_index.insert(std::make_pair(hash, pindexNew)).first; pindexNew->phashBlock = &((*mi).first); BlockMap::iterator miPrev = m_block_index.find(block.hashPrevBlock); - if (miPrev != m_block_index.end()) - { + if (miPrev != m_block_index.end()) { pindexNew->pprev = (*miPrev).second; pindexNew->nHeight = pindexNew->pprev->nHeight + 1; pindexNew->BuildSkip(); @@ -112,7 +112,7 @@ void BlockManager::PruneOneBlockFile(const int fileNumber) // m_blocks_unlinked or setBlockIndexCandidates. auto range = m_blocks_unlinked.equal_range(pindex->pprev); while (range.first != range.second) { - std::multimap::iterator _it = range.first; + std::multimap::iterator _it = range.first; range.first++; if (_it->second == pindex) { m_blocks_unlinked.erase(_it); @@ -207,17 +207,19 @@ void BlockManager::FindFilesToPrune(std::set& setFilesToPrune, uint64_t nPr nLastBlockWeCanPrune, count); } -CBlockIndex * BlockManager::InsertBlockIndex(const uint256& hash) +CBlockIndex* BlockManager::InsertBlockIndex(const uint256& hash) { AssertLockHeld(cs_main); - if (hash.IsNull()) + if (hash.IsNull()) { return nullptr; + } // Return existing BlockMap::iterator mi = m_block_index.find(hash); - if (mi != m_block_index.end()) + if (mi != m_block_index.end()) { return (*mi).second; + } // Create new CBlockIndex* pindexNew = new CBlockIndex(); @@ -236,10 +238,9 @@ bool BlockManager::LoadBlockIndex( } // Calculate nChainWork - std::vector > vSortedByHeight; + std::vector> vSortedByHeight; vSortedByHeight.reserve(m_block_index.size()); - for (const std::pair& item : m_block_index) - { + for (const std::pair& item : m_block_index) { CBlockIndex* pindex = item.second; vSortedByHeight.push_back(std::make_pair(pindex->nHeight, pindex)); } @@ -265,8 +266,7 @@ bool BlockManager::LoadBlockIndex( } } - for (const std::pair& item : vSortedByHeight) - { + for (const std::pair& item : vSortedByHeight) { if (ShutdownRequested()) return false; CBlockIndex* pindex = item.second; pindex->nChainWork = (pindex->pprev ? pindex->pprev->nChainWork : 0) + GetBlockProof(*pindex); @@ -329,8 +329,9 @@ bool BlockManager::LoadBlockIndex( if (pindex->nStatus & BLOCK_FAILED_MASK && (!chainman.m_best_invalid || pindex->nChainWork > chainman.m_best_invalid->nChainWork)) { chainman.m_best_invalid = pindex; } - if (pindex->pprev) + if (pindex->pprev) { pindex->BuildSkip(); + } if (pindex->IsValid(BLOCK_VALID_TREE) && (pindexBestHeader == nullptr || CBlockIndexWorkComparator()(pindexBestHeader, pindex))) pindexBestHeader = pindex; } @@ -338,7 +339,8 @@ bool BlockManager::LoadBlockIndex( return true; } -void BlockManager::Unload() { +void BlockManager::Unload() +{ m_blocks_unlinked.clear(); for (const BlockMap::value_type& entry : m_block_index) { @@ -380,8 +382,7 @@ bool BlockManager::LoadBlockIndexDB(ChainstateManager& chainman) setBlkDataFiles.insert(pindex->nFile); } } - for (std::set::iterator it = setBlkDataFiles.begin(); it != setBlkDataFiles.end(); it++) - { + for (std::set::iterator it = setBlkDataFiles.begin(); it != setBlkDataFiles.end(); it++) { FlatFilePos pos(*it, 0); if (CAutoFile(OpenBlockFile(pos, true), SER_DISK, CLIENT_VERSION).IsNull()) { return false; @@ -390,13 +391,14 @@ bool BlockManager::LoadBlockIndexDB(ChainstateManager& chainman) // Check whether we have ever pruned block & undo files m_block_tree_db->ReadFlag("prunedblockfiles", fHavePruned); - if (fHavePruned) + if (fHavePruned) { LogPrintf("LoadBlockIndexDB(): Block files have previously been pruned\n"); + } // Check whether we need to continue reindexing bool fReindexing = false; m_block_tree_db->ReadReindexing(fReindexing); - if(fReindexing) fReindex = true; + if (fReindexing) fReindex = true; return true; } @@ -405,8 +407,7 @@ CBlockIndex* BlockManager::GetLastCheckpoint(const CCheckpointData& data) { const MapCheckpoints& checkpoints = data.mapCheckpoints; - for (const MapCheckpoints::value_type& i : reverse_iterate(checkpoints)) - { + for (const MapCheckpoints::value_type& i : reverse_iterate(checkpoints)) { const uint256& hash = i.second; CBlockIndex* pindex = LookupBlockIndex(hash); if (pindex) { diff --git a/src/node/blockstorage.h b/src/node/blockstorage.h index ec9f5f791a1..a18203f48d9 100644 --- a/src/node/blockstorage.h +++ b/src/node/blockstorage.h @@ -50,9 +50,8 @@ extern uint64_t nPruneTarget; typedef std::unordered_map BlockMap; -struct CBlockIndexWorkComparator -{ - bool operator()(const CBlockIndex *pa, const CBlockIndex *pb) const; +struct CBlockIndexWorkComparator { + bool operator()(const CBlockIndex* pa, const CBlockIndex* pb) const; }; /** @@ -124,7 +123,8 @@ public: //! Returns last CBlockIndex* that is a checkpoint CBlockIndex* GetLastCheckpoint(const CCheckpointData& data) EXCLUSIVE_LOCKS_REQUIRED(cs_main); - ~BlockManager() { + ~BlockManager() + { Unload(); } };