[libcxx-commits] [libcxx] 582b7d3 - [libc++] Update clang-format style.
Mark de Wever via libcxx-commits
libcxx-commits at lists.llvm.org
Wed Jul 27 09:18:48 PDT 2022
Author: Mark de Wever
Date: 2022-07-27T18:18:44+02:00
New Revision: 582b7d3ff07f5d2fe0c3a7ddf80e0552258d0364
URL: https://github.com/llvm/llvm-project/commit/582b7d3ff07f5d2fe0c3a7ddf80e0552258d0364
DIFF: https://github.com/llvm/llvm-project/commit/582b7d3ff07f5d2fe0c3a7ddf80e0552258d0364.diff
LOG: [libc++] Update clang-format style.
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.
Reviewed By: ldionne, philnik, var-const, #libc
Differential Revision: https://reviews.llvm.org/D129441
Added:
Modified:
libcxx/.clang-format
Removed:
################################################################################
diff --git a/libcxx/.clang-format b/libcxx/.clang-format
index 8d8eff5905af..b5938d53c4fd 100644
--- a/libcxx/.clang-format
+++ b/libcxx/.clang-format
@@ -65,10 +65,9 @@ IndentRequires: true
InsertTrailingCommas: Wrapped
KeepEmptyLinesAtTheStartOfBlocks: false
MaxEmptyLinesToKeep: 1
-NamespaceIndentation: Inner
PackConstructorInitializers: NextLine
-PenaltyIndentedWhitespace: 61
+PenaltyIndentedWhitespace: 2
Language: Cpp
Standard: c++20
More information about the libcxx-commits
mailing list