-
Notifications
You must be signed in to change notification settings - Fork 2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Equihash: Pass each obtained solution to a callback for immediate checking #1146
Equihash: Pass each obtained solution to a callback for immediate checking #1146
Conversation
ACK. |
@zkbot r+ |
📌 Commit 51eb527 has been approved by |
⌛ Testing commit 51eb527 with merge 55f232b... |
Buildbot is hanging on RPC tests because of a bug in this commit. |
} | ||
return CheckProofOfWork(pblock->GetHash(), pblock->nBits, Params().GetConsensus()); | ||
}; | ||
if (EhBasicSolveUncancellable(n, k, curr_state, validBlock)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you spot the problem with this line? 🔎
This is a good example of an easily-overlooked change that can subtly change the execution logic. Or not-so-subtly in this case, thanks to the tests!
Equihash: Pass each obtained solution to a callback for immediate checking Closes #1143
Yes, let's make brackets a requirement on all code. I wish we could lint it. |
☀️ Test successful - zcash |
Closes #1143