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

Eric Liu via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 27 02:20:28 PDT 2016


ioeric marked an inline comment as done.

================
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;
----------------
klimek wrote:
> Looks like we should put this function into some header, so we don't need to re-implement it in every test?
Done. 

Added unittests/Tooling/ReplacementTest.h that contains `ReplacementTest` class and `toReplacements`, both of which are used in multiple test files.


https://reviews.llvm.org/D21748





More information about the cfe-commits mailing list