[PATCH] D75364: [clang-format] Handle macros in function params and return value

Mitchell via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 3 11:20:27 PDT 2020


mitchell-stellar added inline comments.


================
Comment at: clang/unittests/Format/FormatTest.cpp:7359
   verifyIndependentOfContext("bool a = f() && final.f();");
+  verifyFormat("int f(M(x) *p1 = nullptr, M(x) *p2, volatile M(x) *p3);");
+  verifyFormat("M(x) *foo();");
----------------
I would like to see the macro definitions in your tests. It's not clear from looking at just the test line that `M` is supposed to be a macro. It looks like a syntax error.


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

https://reviews.llvm.org/D75364





More information about the cfe-commits mailing list