Page MenuHomePhabricator

Bump to clang-11 and clang-format-11
ClosedPublic

Authored by Fabien on Feb 24 2022, 13:05.

Details

Reviewers
PiRK
Group Reviewers
Restricted Project
Commits
rABC6dcd3c38df1e: Bump to clang-11 and clang-format-11
Summary

This bumps the clang CI version as well as the clang-format linter. We keep both in sync so we prevent inconsistency hazard especially with clang-tidy.

The clang-format update introduce a couple changes:

  • nested ternary operators are now aligned by the colon character
  • Variadic template ellipses have no space anymore, which requires a change to the lint-format-strings.py false positive list.

Depends on D11105.

Test Plan

With Clang:

ninja all check

arc lint --everything

Run the CI builds for the sanitizers.

Diff Detail

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

Event Timeline

@bot build-asan build-ubsan build-tsan

Fabien published this revision for review.Feb 24 2022, 13:49
This revision is now accepted and ready to land.Feb 24 2022, 15:45
This revision was automatically updated to reflect the committed changes.