[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