[clang-format][PR47290] Add ShortNamespaceLines format option
authorKrystian Kuzniarek <krystian.kuzniarek@gmail.com>
Mon, 1 Mar 2021 20:15:04 +0000 (21:15 +0100)
committerBjörn Schäpers <bjoern@hazardy.de>
Mon, 1 Mar 2021 20:28:14 +0000 (21:28 +0100)
commit6ca52815fb3cb32343b9a008fdf9f593da39f203
tree0338d9accd65876b0efcd58e3dfc2ba759bcbc09
parent418b4a7b3158b47043c4c8891eb2e27966a55fa2
[clang-format][PR47290] Add ShortNamespaceLines format option

clang-format documentation states that having enabled
FixNamespaceComments one may expect below code:

c++
namespace a {
foo();
}

to be turned into:

c++
namespace a {
foo();
} // namespace a

In reality, no "// namespace a" was added. The problem was too high
value of kShortNamespaceMaxLines, which is used while deciding whether
a namespace is long enough to be formatted.

As with 9163fe2, clang-format idempotence is preserved.

Differential Revision: https://reviews.llvm.org/D87587
clang/docs/ClangFormatStyleOptions.rst
clang/docs/ReleaseNotes.rst
clang/include/clang/Format/Format.h
clang/lib/Format/Format.cpp
clang/lib/Format/NamespaceEndCommentsFixer.cpp
clang/unittests/Format/NamespaceEndCommentsFixerTest.cpp