Skip to content

Commit

Permalink
Cleanup unused proofCount
Browse files Browse the repository at this point in the history
Summary:
Early exit by tracking proofCount became unworkable in D14663

It is now no longer used.

Test Plan:
  ninja check && ninja check-functional

Reviewers: O1 Bitcoin ABC, #bitcoin_abc, Fabien

Reviewed By: O1 Bitcoin ABC, #bitcoin_abc, Fabien

Differential Revision: https://reviews.bitcoinabc.org/D16410
  • Loading branch information
roqqit committed Jul 8, 2024
1 parent f249751 commit 650634c
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions src/net_processing.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -6505,7 +6505,6 @@ void PeerManagerImpl::ProcessMessage(
return;
}

size_t proofCount = 0;
std::vector<std::pair<avalanche::ProofId, bool>> remoteProofsStatus;
m_avalanche->withPeerManager([&](const avalanche::PeerManager &pm) {
pm.forEachPeer([&](const avalanche::Peer &peer) {
Expand All @@ -6524,8 +6523,6 @@ void PeerManagerImpl::ProcessMessage(
added > 0);
}

proofCount += added;

// In order to properly determine which proof is missing, we
// need to keep scanning for all our proofs.
return true;
Expand Down

0 comments on commit 650634c

Please sign in to comment.