usaxena95 added inline comments.
================
Comment at: clang-tools-extra/clangd/refactor/tweaks/ExtractFunction.cpp:724
+ ReturnStmtVisitor V;
+ for (const auto *RootStmt : ExtZone.RootStmts) {
+ V.TraverseStmt(const_cast<Stmt *>(RootStmt));
----------------
nit: s/auto/Stmt
================
Comment at: clang-tools-extra/clangd/unittests/TweakTests.cpp:613
+ EXPECT_THAT(
+ apply("#define RETURN_IF_ERROR(x) if (x) return\nRETU^RN_IF_ERROR(4);"),
+ StartsWith("unavailable"));
----------------
usaxena95 wrote:
> super super nit: `!x` instead of x.
Oops this is fine. Ignore this comment.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D92408/new/
https://reviews.llvm.org/D92408
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
