[libcxx-commits] [PATCH] D129441: [libc++] Update clang-format style.
Mark de Wever via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Mon Jul 11 11:22:40 PDT 2022
Mordante added a comment.
This change was the only major issue I had with the current style. There are some other things that I don't really like but nothing major. I feel `AlignConsecutiveAssignments` is a bit of a hit or miss, depending on the code it looks great or not really better than without this option. But it doesn't bother me.
I've no strong opinion on `PenaltyIndentedWhitespace` but the new value looks slightly better.
I'll wait for comments of other reviewers and then apply other requested changes.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D129441/new/
https://reviews.llvm.org/D129441
More information about the libcxx-commits
mailing list