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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to