[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