curdeius accepted this revision. curdeius added a comment. It would be great if we had some spec for this style, it gets a bit hard to verify if the formatting is actually what is expected. Having said that, LGTM if the bug reporter confirms the behaviour is ok.
================ Comment at: clang/lib/Format/UnwrappedLineParser.cpp:2248 + + bool RemoveWhiteSmithCaseIndent = + (!Style.IndentCaseBlocks && ---------------- Nit: the style is named `Whitesmiths` so I'd expect the variable to match that: small s in the middle, trailing s. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93806/new/ https://reviews.llvm.org/D93806 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
