[PATCH] D52676: [clang-format] tweaked another case of lambda formatting

[MyDeveloperDay via Phabricator via llvm-commits]

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