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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to