Skip to content

Commit

Permalink
Let ProcessPendingReconstructedRecoveredSigs return void instead of bool
Browse files Browse the repository at this point in the history
Return value is unused and the method actually never returned something.
  • Loading branch information
codablock committed Mar 7, 2019
1 parent f8f867a commit fae33e0
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/llmq/quorums_signing.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -440,7 +440,7 @@ void CSigningManager::CollectPendingRecoveredSigsToVerify(
}
}

bool CSigningManager::ProcessPendingReconstructedRecoveredSigs()
void CSigningManager::ProcessPendingReconstructedRecoveredSigs()
{
decltype(pendingReconstructedRecoveredSigs) l;
{
Expand Down
2 changes: 1 addition & 1 deletion src/llmq/quorums_signing.h
Original file line number Diff line number Diff line change
Expand Up @@ -154,7 +154,7 @@ class CSigningManager
void CollectPendingRecoveredSigsToVerify(size_t maxUniqueSessions,
std::unordered_map<NodeId, std::list<CRecoveredSig>>& retSigShares,
std::unordered_map<std::pair<Consensus::LLMQType, uint256>, CQuorumCPtr, StaticSaltedHasher>& retQuorums);
bool ProcessPendingReconstructedRecoveredSigs();
void ProcessPendingReconstructedRecoveredSigs();
bool ProcessPendingRecoveredSigs(CConnman& connman); // called from the worker thread of CSigSharesManager
void ProcessRecoveredSig(NodeId nodeId, const CRecoveredSig& recoveredSig, const CQuorumCPtr& quorum, CConnman& connman);
void Cleanup(); // called from the worker thread of CSigSharesManager
Expand Down

0 comments on commit fae33e0

Please sign in to comment.