Fixed cppcheck errors

This commit is contained in:
SChernykh
2025-10-01 16:39:42 +02:00
parent 5d7dfa2466
commit e1da9ff4d3
2 changed files with 10 additions and 10 deletions
+8 -8
View File
@@ -406,7 +406,7 @@ bool SideChain::get_shares(const PoolBlock* tip, std::vector<MinerShare>& shares
return false; return false;
} }
PoolBlock* uncle = it->second; const PoolBlock* uncle = it->second;
// Skip uncles which are already out of PPLNS window // Skip uncles which are already out of PPLNS window
if (tip->m_sidechainHeight - uncle->m_sidechainHeight >= m_chainWindowSize) { if (tip->m_sidechainHeight - uncle->m_sidechainHeight >= m_chainWindowSize) {
@@ -747,7 +747,7 @@ bool SideChain::add_block(const PoolBlock& block)
return true; return true;
} }
PoolBlock* SideChain::find_block(const hash& id) const const PoolBlock* SideChain::find_block(const hash& id) const
{ {
ReadLock lock(m_sidechainLock); ReadLock lock(m_sidechainLock);
@@ -759,7 +759,7 @@ PoolBlock* SideChain::find_block(const hash& id) const
return nullptr; return nullptr;
} }
PoolBlock* SideChain::find_block_by_merkle_root(const root_hash& merkle_root) const const PoolBlock* SideChain::find_block_by_merkle_root(const root_hash& merkle_root) const
{ {
ReadLock lock(m_sidechainLock); ReadLock lock(m_sidechainLock);
@@ -834,7 +834,7 @@ bool SideChain::get_outputs_blob(PoolBlock* block, uint64_t total_reward, std::v
auto it = block->m_sidechainId.empty() ? m_blocksById.end() : m_blocksById.find(block->m_sidechainId); auto it = block->m_sidechainId.empty() ? m_blocksById.end() : m_blocksById.find(block->m_sidechainId);
if (it != m_blocksById.end()) { if (it != m_blocksById.end()) {
PoolBlock* b = it->second; const PoolBlock* b = it->second;
const size_t n = b->m_outputs.size(); const size_t n = b->m_outputs.size();
blob.reserve(n * 39 + 64); blob.reserve(n * 39 + 64);
@@ -1521,7 +1521,7 @@ void SideChain::verify(PoolBlock* block)
} }
// If it's invalid then this block is also invalid // If it's invalid then this block is also invalid
PoolBlock* parent = it->second; const PoolBlock* parent = it->second;
if (parent->m_invalid) { if (parent->m_invalid) {
block->m_verified = true; block->m_verified = true;
block->m_invalid = true; block->m_invalid = true;
@@ -1570,7 +1570,7 @@ void SideChain::verify(PoolBlock* block)
if (!block->m_uncles.empty()) { if (!block->m_uncles.empty()) {
mined_blocks.reserve(UNCLE_BLOCK_DEPTH * 2 + 1); mined_blocks.reserve(UNCLE_BLOCK_DEPTH * 2 + 1);
PoolBlock* tmp = parent; const PoolBlock* tmp = parent;
for (uint64_t i = 0, n = std::min<uint64_t>(UNCLE_BLOCK_DEPTH, block->m_sidechainHeight + 1); tmp && (i < n); ++i) { for (uint64_t i = 0, n = std::min<uint64_t>(UNCLE_BLOCK_DEPTH, block->m_sidechainHeight + 1); tmp && (i < n); ++i) {
mined_blocks.push_back(tmp->m_sidechainId); mined_blocks.push_back(tmp->m_sidechainId);
mined_blocks.insert(mined_blocks.end(), tmp->m_uncles.begin(), tmp->m_uncles.end()); mined_blocks.insert(mined_blocks.end(), tmp->m_uncles.begin(), tmp->m_uncles.end());
@@ -1606,7 +1606,7 @@ void SideChain::verify(PoolBlock* block)
return; return;
} }
PoolBlock* uncle = it->second; const PoolBlock* uncle = it->second;
// If it's invalid then this block is also invalid // If it's invalid then this block is also invalid
if (uncle->m_invalid) { if (uncle->m_invalid) {
@@ -2041,7 +2041,7 @@ void SideChain::update_depths(PoolBlock* block)
if (it == m_blocksByHeight.end()) { if (it == m_blocksByHeight.end()) {
continue; continue;
} }
for (PoolBlock* child : it->second) { for (const PoolBlock* child : it->second) {
if (child->m_parent == block->m_sidechainId) { if (child->m_parent == block->m_sidechainId) {
if (i != 1) { if (i != 1) {
LOGWARN(3, "Block " << block->m_sidechainId << ": m_sidechainHeight is inconsistent with child's m_sidechainHeight."); LOGWARN(3, "Block " << block->m_sidechainId << ": m_sidechainHeight is inconsistent with child's m_sidechainHeight.");
+2 -2
View File
@@ -53,8 +53,8 @@ public:
[[nodiscard]] bool add_block(const PoolBlock& block); [[nodiscard]] bool add_block(const PoolBlock& block);
void get_missing_blocks(unordered_set<hash>& missing_blocks) const; void get_missing_blocks(unordered_set<hash>& missing_blocks) const;
[[nodiscard]] PoolBlock* find_block(const hash& id) const; [[nodiscard]] const PoolBlock* find_block(const hash& id) const;
[[nodiscard]] PoolBlock* find_block_by_merkle_root(const root_hash& merkle_root) const; [[nodiscard]] const PoolBlock* find_block_by_merkle_root(const root_hash& merkle_root) const;
void watch_mainchain_block(const ChainMain& data, const hash& possible_merkle_root); void watch_mainchain_block(const ChainMain& data, const hash& possible_merkle_root);
[[nodiscard]] const PoolBlock* get_block_blob(const hash& id, std::vector<uint8_t>& blob) const; [[nodiscard]] const PoolBlock* get_block_blob(const hash& id, std::vector<uint8_t>& blob) const;