ilya-biryukov accepted this revision. ilya-biryukov added a comment. This revision is now accepted and ready to land.
LGTM ================ Comment at: clang/include/clang/Tooling/Refactoring/SourceCode.h:80 +llvm::Optional<CharSourceRange> +getRangeForEdit(const CharSourceRange &EditRange, const ASTContext &Context); } // namespace tooling ---------------- NIT: I can see why passing two structs instead of a single `ASTContext` is annoying, but still suggest adding an overload that accepts a `SourceManager` and `LangOptions`. (The current overload would trivially call into the other one). The reason is that `ASTContext` is not available where you would need this function. In any case, having the `ASTContext` overload is convenient. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64924/new/ https://reviews.llvm.org/D64924 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
