[libcxx-commits] [PATCH] D99691: [libcxx] adjusts formatting rules

[Christopher Di Bella via Phabricator via libcxx-commits]

cjdb added inline comments.


================
Comment at: libcxx/.clang-format:16
+BreakBeforeConceptDeclarations: true
+IndentRequires: false
 ---
----------------
Quuxplusone wrote:
> Please set "IndentRequires" to true. Or at least, regardless of what tools you run locally to format your code, please ensure that what gets checked in looks like
> 
> ```
> template <class _Tp>
>   requires foo<_Tp>
> inline constexpr bar(_Tp) {
> ```
> and not
> ```
> template <class _Tp>
> requires foo<_Tp>
> inline constexpr bar(_Tp) {
> ```
The current libc++ style is as if `IndentRequires` is `false` and I'd prefer to keep it that way; unless @ldionne asks it to be changed.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D99691/new/

https://reviews.llvm.org/D99691