[PATCH] D87587: [clang-format][PR47290] Add ShortNamespaceLines format option
Krystian Kuzniarek via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Dec 21 16:34:04 PST 2020
kuzkry updated this revision to Diff 313218.
kuzkry retitled this revision from "[clang-format][PR47290] Add MaxUnwrappedLinesForShortNamespace format option" to "[clang-format][PR47290] Add ShortNamespaceLines format option".
kuzkry added a comment.
Renamed MaxUnwrappedLinesForShortNamespace to ShortNamespaceLines (thank you @MyDeveloperDay for the idea and @curdeius for mentioning clang-tidy). I haven't changed the description of this option so you might wanna complain on that. Tbh. English isn't my native language so I'll take your tips if you wanna help simplify it.
Luckily formatting is magically fixed and hopefully CI check's gonna be satisfied.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D87587/new/
https://reviews.llvm.org/D87587
Files:
clang/docs/ClangFormatStyleOptions.rst
clang/include/clang/Format/Format.h
clang/lib/Format/Format.cpp
clang/lib/Format/NamespaceEndCommentsFixer.cpp
clang/unittests/Format/NamespaceEndCommentsFixerTest.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D87587.313218.patch
Type: text/x-patch
Size: 10452 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20201222/464a0546/attachment-0001.bin>
More information about the cfe-commits
mailing list