owenpan added inline comments.
================ Comment at: clang/unittests/Format/FormatTest.cpp:2986-2990 + verifyFormat("#define X " + " \\\n" + " case 0: break;\n" + "#include \"f\"\n", + Style); ---------------- We can move it down to take advantage of the shorter `ColumnLimit`. ================ Comment at: clang/unittests/Format/FormatTest.cpp:3006 Style); Style.ColumnLimit = 21; verifyFormat("switch (a) {\n" ---------------- We usually don't append a newline at the end of a test case. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D135422/new/ https://reviews.llvm.org/D135422 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits