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
refactor: Refactor CalcCbTxMerkleRootQuorums
#4954
Conversation
This pull request has conflicts, please rebase. |
bfa9ea4
to
5a8e060
Compare
f88dfa3
to
347ce3e
Compare
This pull request has conflicts, please rebase. |
347ce3e
to
1127dcd
Compare
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.
utACK for squash merge
CalcCbTxMerkleRootQuorums
CalcCbTxMerkleRootQuorums
Ofc it's not 0 when cache is used... not sure what I was thinking about.
I messed up (818886a) 🙈 pls re-review |
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.
utACK for squash merge
Picking suggestions from #4952 (comment), applying few cleanups and doing some trivial-ish refactoring on top. Also moved few things including
IsQuorumRotationEnabled
out of the inner loop - should not result in any changes in behaviour, could simply be a tiny bit faster.builds on top of #4952