[PATCH] D110481: fixes bug #51926 where dangling comma caused overrun
Björn Schäpers via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sat Sep 25 11:37:30 PDT 2021
HazardyKnusperkeks added inline comments.
================
Comment at: clang/unittests/Format/FormatTest.cpp:17805
+ " Section{\n"
+ " 0, bar(),\n"
+ " }\n"
----------------
Here is no alignment, maybe add another case with another line?
What happens if one line has the trailing comma and the next doesn't?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D110481/new/
https://reviews.llvm.org/D110481
More information about the cfe-commits
mailing list