[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