[PATCH] D52676: [clang-format] tweaked another case of lambda formatting
Daniel Jasper via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Oct 30 07:28:18 PDT 2018
djasper added a comment.
I think this roughly looks fine. Krasimir, any thoughts?
================
Comment at: unittests/Format/FormatTest.cpp:11854
+ // case above.
+ {
+ auto Style = getGoogleStyle();
----------------
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.
================
Comment at: unittests/Format/FormatTest.cpp:11865
+ }
+ {
+ verifyFormat("SomeFunction(\n"
----------------
No need for this scope.
Repository:
rC Clang
https://reviews.llvm.org/D52676
More information about the cfe-commits
mailing list