[PATCH] D87587: [clang-format][PR47290] Add ShortNamespaceLines format option

Krystian Kuzniarek via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 12 04:22:05 PST 2021


kuzkry marked 2 inline comments as done.
kuzkry added inline comments.


================
Comment at: clang/unittests/Format/NamespaceEndCommentsFixerTest.cpp:1162
+
+  EXPECT_EQ(DefaultUnwrappedLines, Style.ShortNamespaceLines);
+  EXPECT_EQ("namespace ShortNamespace {\n"
----------------
curdeius wrote:
> Nit.
Well, instead of using comments or giving it a separate test, I wanted to have a nice self-explanatory name.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D87587/new/

https://reviews.llvm.org/D87587



More information about the cfe-commits mailing list