owenpan accepted this revision.
owenpan added inline comments.
================
Comment at: clang/unittests/Format/TokenAnnotatorTest.cpp:752
+TEST_F(TokenAnnotatorTest, UnderstandsTrailingReturnTypeAutoInLabmdas) {
+ auto Tokens = annotate("[]() -> auto {}");
----------------
HazardyKnusperkeks wrote:
> I'd just merge it with the test above.
> I'd just merge it with the test above.
+1.
================
Comment at: clang/unittests/Format/TokenAnnotatorTest.cpp:753-756
+ auto Tokens = annotate("[]() -> auto {}");
+ ASSERT_EQ(Tokens.size(), 9u) << Tokens;
+ EXPECT_TOKEN(Tokens[4], tok::arrow, TT_LambdaArrow);
+ EXPECT_TOKEN(Tokens[6], tok::l_brace, TT_LambdaLBrace);
----------------
Should we add a test case with an `&` between `auto` and `{`?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D130299/new/
https://reviews.llvm.org/D130299
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
