[PATCH] D21748: Implement tooling::Replacements as a class.

Manuel Klimek via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 26 07:20:18 PDT 2016


klimek added inline comments.

================
Comment at: unittests/Tooling/RefactoringTest.cpp:112
@@ -111,4 +111,3 @@
 
-TEST_F(ReplacementTest, CanApplyReplacements) {
-  FileID ID = Context.createInMemoryFile("input.cpp",
-                                         "line1\nline2\nline3\nline4");
+static Replacements toReplacements(const std::set<Replacement> &Replaces) {
+  Replacements Result;
----------------
Looks like we should put this function into some header, so we don't need to re-implement it in every test?


https://reviews.llvm.org/D21748





More information about the cfe-commits mailing list