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