[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