Avoid an extraneous dispatch to the same context we're already in.

Get rid of stored proposals in a more sane way.
This commit is contained in:
JoelKatz
2013-06-13 23:38:21 -07:00
parent a349b9bcd0
commit 968768cdb8
2 changed files with 3 additions and 5 deletions

View File

@@ -1043,6 +1043,8 @@ void LedgerConsensus::accept(SHAMap::ref set, LoadEvent::pointer)
boost::recursive_mutex::scoped_lock masterLock(theApp->getMasterLock());
assert(set->getHash() == mOurPosition->getCurrentHash());
theApp->getOPs().peekStoredProposals().clear(); // these are now obsolete
uint32 closeTime = roundCloseTime(mOurPosition->getCloseTime());
bool closeTimeCorrect = true;
if (closeTime == 0)

View File

@@ -1100,11 +1100,7 @@ static void checkPropose(Job& job, boost::shared_ptr<ripple::TMProposeSet> packe
}
if (isTrusted)
{
theApp->getJobQueue().addJob(jtPROPOSAL_t, "trustedProposal",
BIND_TYPE(&NetworkOPs::processTrustedProposal, &theApp->getOPs(),
proposal, packet, nodePublic, prevLedger, sigGood));
}
theApp->getOPs().processTrustedProposal(proposal, packet, nodePublic, prevLedger, sigGood);
else if (sigGood && (prevLedger == consensusLCL))
{ // relay untrusted proposal
WriteLog (lsTRACE, Peer) << "relaying untrusted proposal";