kadircet accepted this revision. kadircet added a comment. This revision is now accepted and ready to land.
Thanks, LGTM! ================ Comment at: clang-tools-extra/clangd/unittests/tweaks/ExpandAutoTypeTests.cpp:68 + EXPECT_EQ(apply("ns::Class * foo() { au^to c = foo(); }"), + "ns::Class * foo() { ns::Class * c = foo(); }"); ---------------- can you also test for shortening? e.g. ``` void ns::Func() { aut^o x = new ns::Class::Nested{}; } ``` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94259/new/ https://reviews.llvm.org/D94259 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits