[Bot Message]
One or more PR numbers were detected in the summary.
Links to those PRs have been inserted into the summary for reference.
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
All Stories
Jul 15 2020
[Bot Message]
One or more PR numbers were detected in the summary.
Links to those PRs have been inserted into the summary for reference.
[Bot Message]
One or more PR numbers were detected in the summary.
Links to those PRs have been inserted into the summary for reference.
Install base-devel instead of clang
used fuzzed input for the prefix as well, that increased coverage a bit
Use the development meta group instead: base-devel. This is somewhat the equivalent for the Debian build-essential and should already be installed on most Arch boxes.
Jul 14 2020
[Bot Message]
One or more PR numbers were detected in the summary.
Links to those PRs have been inserted into the summary for reference.
Rebase on linter fixes...
[Bot Message]
One or more PR numbers were detected in the summary.
Links to those PRs have been inserted into the summary for reference.
Jul 13 2020
Rebased on the partial backports. This patch should only be the remaining
changes directly related to ValidationState.
missed a few callsites, fixed a log output to point to the right place
Hm, since the private function set the CChainState pointer for the finalized block, it's more fitting for it to get the "Mark" name. The public facing API should be FinalizeBlock then
Simple name change are required, but otherwise, it looks good.
now that FinalizeBlockInternal is a private member of CChainState and has its own declaration, the lock annotation can go there instead of its definition
missed a spurious EXCLUSIVE_LOCKS_REQUIRED on CChainState::FinalizeBlock definition
It appears I have gotten ahead of myself and overcomplicated this diff. The previous diff changed behavior in a way that isn't needed until https://github.com/bitcoin/bitcoin/pull/15971. That one comes next and will incorporate the feedback given so far.
[Bot Message]
One or more PR numbers were detected in the summary.
Links to those PRs have been inserted into the summary for reference.