[PATCH] D32480: clang-format: Add CompactNamespaces option

[Manuel Klimek via Phabricator via cfe-commits]

klimek added a comment.

Generally LG from my side.



================
Comment at: unittests/Format/FormatTest.cpp:1363-1381
+  EXPECT_EQ("namespace aaaaaaaaaaaaaaaaaaaaaaaaaaaa {\n"
+			"namespace bbbbbbbbbbbbbbbbbbbbbbbbbbbb {\n"
+            "}} // namespace aaaaaaaaaaaaaaaaaaaaaaaaaaaa::bbbbbbbbbbbbbbbbbbbbbbbbbbbb",
+            format("namespace aaaaaaaaaaaaaaaaaaaaaaaaaaaa {\n"
+                   "namespace bbbbbbbbbbbbbbbbbbbbbbbbbbbb {\n"
+                   "} // namespace bbbbbbbbbbbbbbbbbbbbbbbbbbbb\n"
+                   "} // namespace aaaaaaaaaaaaaaaaaaaaaaaaaaaa",
----------------
These tests become more readable if you set up a style with a smaller column limit.


https://reviews.llvm.org/D32480