hokein marked an inline comment as done. hokein added inline comments.
================ Comment at: clangd/refactor/Tweak.h:81 + /// EXPECTS: prepare() was called and returned true. + virtual Expected<tooling::Replacements> execute(const Selection &Sel) = 0; }; ---------------- I think the current name is slightly better than `doApply`, I also have a few candidates in mind `perform`, `invoke`. Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57739/new/ https://reviews.llvm.org/D57739 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
