[PATCH] D88956: [clang-format] Fix misformatted macro definitions after D86959

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 7 09:31:09 PDT 2020


MyDeveloperDay added inline comments.


================
Comment at: clang/unittests/Format/FormatTest.cpp:8365
+  EXPECT_TOKEN(Tokens[11], tok::star, TT_PointerOrReference);
+  verifyFormat("#define lambda [](const decltype(x) *ptr) {}");
+}
----------------
I like this style of test, I think sometimes even the tests seem to work even if the type is incorrectly determined, but generally that leads to problems further down the road. I can see this construct being useful


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88956



More information about the cfe-commits mailing list