[libcxx-commits] [PATCH] D129441: [libc++] Update clang-format style.

[Mark de Wever via Phabricator via libcxx-commits]

Mordante created this revision.
Mordante added reviewers: EricWF, ldionne, philnik, var-const.
Herald added a project: All.
Mordante requested review of this revision.
Herald added a project: libc++.
Herald added a subscriber: libcxx-commits.
Herald added a reviewer: libc++.

After evaluating the new style I noticed inner namespaces are now
indented. I am not fond of that style and I've seen some other review
comment in this regard so I propose we remove this option and use the
LLVM default not to indent it.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D129441

Files:
  libcxx/.clang-format


Index: libcxx/.clang-format
===================================================================
--- libcxx/.clang-format
+++ libcxx/.clang-format
@@ -65,7 +65,6 @@
 InsertTrailingCommas: Wrapped
 KeepEmptyLinesAtTheStartOfBlocks: false
 MaxEmptyLinesToKeep: 1
-NamespaceIndentation: Inner
 PackConstructorInitializers: NextLine
 
 PenaltyIndentedWhitespace: 61


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D129441.443494.patch
Type: text/x-patch
Size: 361 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20220710/760ccd29/attachment.bin>