[PATCH] D12362: [clang-format] Add support of consecutive declarations alignment

Daniel Jasper via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 26 08:07:55 PDT 2015


djasper added inline comments.

================
Comment at: unittests/Format/FormatTest.cpp:8619
@@ -8618,1 +8618,3 @@
 
+TEST_F(FormatTest, AlignConsecutiveDeclarations) {
+  FormatStyle Alignment = getLLVMStyle();
----------------
This needs tests that check what happens if both declarations and assignments are aligned.

================
Comment at: unittests/Format/FormatTest.cpp:8703
@@ +8702,3 @@
+  verifyFormat("#define A                                                      "
+               "                \\\n"
+               "  int       aaaa = 12;                                         "
----------------
Reduce Alignment.ColumnLimit so that tests don't need line-wraps.


http://reviews.llvm.org/D12362





More information about the cfe-commits mailing list