Page MenuHomePhabricator

[Part 2 of 5] Add a CChainState class to clarify internal interfaces
ClosedPublic

Authored by jasonbcox on Oct 24 2018, 18:50.

Details

Summary

Move block writing out of AcceptBlock

Backport of Core PR 10279 commit e104f0f
https://github.com/bitcoin/bitcoin/pull/10279/commits/e104f0fb7e464f8cadb4cbc99e8329d22c35c36a
Depends on D1968
Progress towards T572

Test Plan

make check
test_runner.py

Diff Detail

Repository
rABC Bitcoin ABC
Branch
arcpatch-D1969
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 5320
Build 8702: Bitcoin ABC Buildbot (legacy)
Build 8701: arc lint + arc unit

Event Timeline

Rebase, remove excess whitespace change

jasonbcox requested changes to this revision.Nov 30 2018, 22:43
jasonbcox added a subscriber: jasonbcox.
jasonbcox added inline comments.
src/validation.cpp
3176 ↗(On Diff #6183)

Missing lines from e104f0f

3879 ↗(On Diff #6183)

is this equivalent to return false; as in the original PR?

4815 ↗(On Diff #6183)

This line was moved in the original PR

This revision now requires changes to proceed.Nov 30 2018, 22:43
jasonbcox edited reviewers, added: schancel; removed: jasonbcox.
jasonbcox edited the summary of this revision. (Show Details)
jasonbcox edited the test plan for this revision. (Show Details)

Rebase + fixes described above

This revision is now accepted and ready to land.Mar 28 2019, 10:12
This revision was automatically updated to reflect the committed changes.