[libcxx-commits] [PATCH] D92229: [libc++] Update clang-format configuration

Marek Kurdej via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Fri Nov 27 06:58:04 PST 2020


curdeius created this revision.
curdeius added reviewers: ldionne, EricWF, Mordante, miscco, cjdb.
Herald added subscribers: libcxx-commits, arichardson.
Herald added a project: libc++.
Herald added a reviewer: libc++.
curdeius requested review of this revision.

I tried to match at least partially current state of the code.
However, the libc++ codebase is pretty much inconsistent everywhere.
As discussed with @ldionne in one of the reviews, the idea would be to agree on a common style once and for all and then just apply it.
(Apparently we could reformat the whole codebase, probably step by step, adding the formatting commits to "don't blame" list.)

Also updated: libcxx/utils/generate_feature_test_macro_components.py script. Adjusted to match the formatting. Added comments to turn off further formatting and avoid linter warnings in reviews.

I added as reviewers those that work on new features (and so new headers) like <concepts>, <ranges> and <format>, as it would be probably the best place to test and tweak the formatting configuration.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D92229

Files:
  libcxx/.clang-format
  libcxx/utils/generate_feature_test_macro_components.py

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D92229.308042.patch
Type: text/x-patch
Size: 4944 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20201127/c9bac466/attachment.bin>


More information about the libcxx-commits mailing list