Page MenuHomePhabricator

Update clang-format to 4.0
ClosedPublic

Authored by deadalnix on Jan 3 2018, 21:12.

Details

Summary

As per title. It seems like mst software out there ship with 4.0 nowadays, and we just released, so it's a good time to upgrade.

Test Plan
make check

Diff Detail

Repository
rABC Bitcoin ABC
Branch
clangfmt4
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 1506
Build 1506: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Jan 4 2018, 13:54
schancel requested changes to this revision.Jan 4 2018, 13:57
This revision now requires changes to proceed.Jan 4 2018, 13:57

brew only has clang-format@3.8 and clang-format version 6.0.0 (tags/google/stable/2017-11-14). There's substantial changes on 6.0.0 from this diff.

Backport some formating from more recent version of clang-format

Is this now clang-format 6 or something? I don't understand the comment.

This revision is now accepted and ready to land.Jan 7 2018, 04:17
This revision was automatically updated to reflect the committed changes.