[PATCH] D135422: [clang-format] Fix misattributing preprocessor directives to macros
Owen Pan via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Oct 10 01:47:22 PDT 2022
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
More information about the cfe-commits
mailing list