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

Oleg Smolsky via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 31 08:39:13 PDT 2018


oleg.smolsky marked 2 inline comments as done.
oleg.smolsky added inline comments.


================
Comment at: unittests/Format/FormatTest.cpp:11854
+  // case above.
+  {
+    auto Style = getGoogleStyle();
----------------
djasper wrote:
> No need to use a scope here. Feel free to redefine Style.
> 
> If in fact you feel like that is getting out of hand, maybe extract a separate TEST_F() for this.
OK, sure, removed the scoped block.


================
Comment at: unittests/Format/FormatTest.cpp:11865
+  }
+  {
+    verifyFormat("SomeFunction(\n"
----------------
djasper wrote:
> No need for this scope.
Oh, right, that's vestigial. Removed.


Repository:
  rC Clang

https://reviews.llvm.org/D52676





More information about the cfe-commits mailing list