HazardyKnusperkeks added inline comments.
================
Comment at: clang/unittests/Format/FormatTest.cpp:4810-4811
+
+ verifyFormat("#define STRINGIFY(t) #t\n"
+ "#define MAKEVERSIONSTRING(x, y, z, build) STRINGIFY(x) \".\"
STRINGIFY(y) \".\" STRINGIFY(z) \".\" STRINGIFY(build)\n",
+ ZeroColumn);
----------------
curdeius wrote:
> Please test with something simpler.
Did this show the bugged behavior without the patch?
Regarding the `// clang-format off` there is the question what do we want in
the tests? Show the long lines as the long lines, then we need to turn it off
some times. Or do we want to keep the limit and break the strings, thus making
it harder to read for the human?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D116859/new/
https://reviews.llvm.org/D116859
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
