[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