Page MenuHomePhabricator

Resolved forgotten chain.h merge.
ClosedPublic

Authored by nakihito on Sep 4 2018, 22:29.

Details

Reviewers
jasonbcox
deadalnix
schancel
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Commits
rSTAGING5977efad639b: Resolved forgotten chain.h merge.
rABC5977efad639b: Resolved forgotten chain.h merge.
Summary

Continues work on T380, dependent on D1656. Fixes D1659. Removed CBlockStatus class from chain.h and placed into its own .h file, blockstatus.h. Updated other .h and .cpp files for dependencies.

Test Plan

make check

Diff Detail

Repository
rABC Bitcoin ABC
Branch
SeperatingBlockStatusClassFix
Lint
Lint Passed
SeverityLocationCodeMessage
Auto-Fixsrc/validation.cpp:1CFMTCode style violation
Unit
No Test Coverage
Build Status
Buildable 3193
Build 4473: Bitcoin ABC Buildbot (legacy)
Build 4472: arc lint + arc unit

Event Timeline

Owners added a reviewer: Restricted Owners Package.Sep 4 2018, 22:29
jasonbcox requested changes to this revision.Sep 5 2018, 04:12
jasonbcox added inline comments.
src/validation.cpp
598

This is strange and shouldn't happen with clang-format-4.0. But I set you up with 4.0, so not sure what happened here. Please revert this.

This revision now requires changes to proceed.Sep 5 2018, 04:12
src/validation.cpp
598

I can change this back, but there are a similar couple of lines from 567-573 that looks more like this changed format than the previous format. In terms of consistency, I believe this change is correct, but am happy to revert to the previous.

Fixed validation.cpp formatting error.

This revision is now accepted and ready to land.Sep 7 2018, 14:12
This revision was automatically updated to reflect the committed changes.