Page MenuHomePhabricator

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

Authored by schancel on Oct 24 2018, 18:49.

Details

Summary

Move txindex/undo data disk location stuff out of ConnectBlock

Backport of Core PR10279 commit 50701ba

T572

Test Plan
make VERBOSE=1 check && ./test/functional/test_runner.py

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Oct 24 2018, 19:31
deadalnix requested changes to this revision.Oct 24 2018, 20:43
deadalnix added a subscriber: deadalnix.

Please edit the description message to have PR10279 instead of in two words so that's easy to grep for.

src/validation.cpp
1458 ↗(On Diff #5523)

Braces.

1482 ↗(On Diff #5523)

dito

1483 ↗(On Diff #5523)

dito

This revision now requires changes to proceed.Oct 24 2018, 20:43
deadalnix requested changes to this revision.Oct 24 2018, 22:47
deadalnix added inline comments.
src/validation.cpp
1746 ↗(On Diff #5529)

Why does this stays ?

1755 ↗(On Diff #5529)

dito

This revision now requires changes to proceed.Oct 24 2018, 22:47
src/validation.cpp
1455 ↗(On Diff #5529)

braces

src/validation.cpp
1479 ↗(On Diff #5529)

GetId

Remove stray unused variables

deadalnix requested changes to this revision.Oct 25 2018, 22:11
deadalnix added inline comments.
src/validation.cpp
1455 ↗(On Diff #5530)

braces

This revision now requires changes to proceed.Oct 25 2018, 22:11
This revision is now accepted and ready to land.Nov 29 2018, 00:52
This revision was automatically updated to reflect the committed changes.