[PATCH] D125162: [clang-format] fix alignment w/o binpacked args

cha5on via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 9 19:19:00 PDT 2022


cha5on marked 3 inline comments as done.
cha5on added inline comments.


================
Comment at: clang/unittests/Format/FormatTest.cpp:17321
+
+    verifyFormat(Expected, ToFormat, Alignment);
+  }
----------------
curdeius wrote:
> I don't think you need to create variables, just online the strings.
> 
> Also, could you minimize (and possibly split) as much as possible these test cases?
Shrank this down and added a second testcase for direct-list-initialization.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D125162/new/

https://reviews.llvm.org/D125162



More information about the cfe-commits mailing list