Skip to content

Commit

Permalink
[libc++] Update clang-format style.
Browse files Browse the repository at this point in the history
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
  • Loading branch information
mordante committed Jul 27, 2022
1 parent ebee4fb commit 582b7d3
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions libcxx/.clang-format
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit 582b7d3

Please sign in to comment.