MyDeveloperDay added a comment.
I think if you have a proposal for changing the behavior lets see the patch and
how it impacts the existing unit tests
something tells me these tests are going to change?
// A lambda passed as arg0 is always pushed to the next line.
verifyFormat("SomeFunction(\n"
" [this] {\n"
" //\n"
" },\n"
" 1);\n");
// A multi-line lambda passed as arg1 forces arg0 to be pushed out, just like
the arg0
// case above.
auto Style = getGoogleStyle();
Style.BinPackArguments = false;
verifyFormat("SomeFunction(\n"
" a,\n"
" [this] {\n"
" //\n"
" },\n"
" b);\n",
Style);
verifyFormat("SomeFunction(\n"
" a,\n"
" [this] {\n"
" //\n"
" },\n"
" b);\n");
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D52676/new/
https://reviews.llvm.org/D52676
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
