Page MenuHomePhabricator

[2 of 5] validation: Extract basic block file logic into FlatFileSeq class.
ClosedPublic

Authored by markblundeberg on Jul 7 2019, 21:55.

Details

Summary

partial PR15118 backport https://github.com/bitcoin/bitcoin/pull/15118/commits

This combines four refactor commits:
https://github.com/bitcoin/bitcoin/compare/9183d6ef6~1..e0380933e
...and a commit adding unit tests:
https://github.com/bitcoin/bitcoin/pull/15118/commits/4c01e4e159db82ce4b2acce75f709cac996367d7

These have been squashed due to intermediate commits not even building
properly. The tests have been modified to use CDiskBlockPos as the
rename is planned for a subsequent diff. Otherwise, it's the same.

Depends on D3579 D3583 D3584 D3585

Test Plan

make check

Diff Detail

Repository
rABC Bitcoin ABC
Branch
PR15118b
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 6770
Build 11587: Bitcoin ABC Buildbot (legacy)
Build 11586: arc lint + arc unit

Event Timeline

markblundeberg retitled this revision from [part 2 of 5] validation: Extract basic block file logic into FlatFileSeq class. to [2 of 5] validation: Extract basic block file logic into FlatFileSeq class..Jul 7 2019, 21:55
markblundeberg added inline comments.
src/Makefile.am
255 ↗(On Diff #10063)

oops, this needs to go in CMakeLists too

CI failure from validation_block_tests, seems to be unrelated...

rebase because all parents are landed

jasonbcox requested changes to this revision.Jul 10 2019, 19:26

Looks good otherwise.

src/flatfile.cpp
24

brackets

75

comment on previous line

This revision now requires changes to proceed.Jul 10 2019, 19:26
deadalnix added inline comments.
src/flatfile.cpp
5 ↗(On Diff #10276)

Standard lib include come last.

for include order (and rebuild)

This revision is now accepted and ready to land.Jul 16 2019, 14:50