summaryrefslogtreecommitdiffstats
path: root/clang-tools-extra/unittests/cpp11-migrate/TransformTest.cpp
diff options
context:
space:
mode:
authorEdwin Vane <edwin.vane@intel.com>2013-06-18 15:31:01 +0000
committerEdwin Vane <edwin.vane@intel.com>2013-06-18 15:31:01 +0000
commit62c013db6ce8dabac58a313c5e4ca05e75e45881 (patch)
treebb79b5905ac9a479c6f8b9b7c38bdb3a859fcfc3 /clang-tools-extra/unittests/cpp11-migrate/TransformTest.cpp
parent302c0da6f138d1475b497a6e987787e94f15046f (diff)
downloadbcm5719-llvm-62c013db6ce8dabac58a313c5e4ca05e75e45881.tar.gz
bcm5719-llvm-62c013db6ce8dabac58a313c5e4ca05e75e45881.zip
cpp11-migrate: Transform now responsible for applying replacements
To make it possible for replacements made to headers as part of transforming one translation unit to not be visible to the transform of other translation units, Transform now handles replacement application as part of its end-of-source handling. Several things were simplified as a result: - The duplicated code in every transform for applying replacements is now gone and replaced with one location in Transform. - RefactoringTool is no longer used since Transform houses the Replacements structure. - RewriterContainer is now a private implementation detail of Transform (also renamed to RewriterManager since its behaviour is slightly different now with respect to lifetime of objects). - There's now no distinction between input and output file state. Misc notes: - Interface changes reflected in unit tests. - Replacements for files other than the main file are assumed to be for headers and stored as such. llvm-svn: 184194
Diffstat (limited to 'clang-tools-extra/unittests/cpp11-migrate/TransformTest.cpp')
-rw-r--r--clang-tools-extra/unittests/cpp11-migrate/TransformTest.cpp7
1 files changed, 3 insertions, 4 deletions
diff --git a/clang-tools-extra/unittests/cpp11-migrate/TransformTest.cpp b/clang-tools-extra/unittests/cpp11-migrate/TransformTest.cpp
index eeadbf1ecba..af8a7a703ac 100644
--- a/clang-tools-extra/unittests/cpp11-migrate/TransformTest.cpp
+++ b/clang-tools-extra/unittests/cpp11-migrate/TransformTest.cpp
@@ -14,10 +14,9 @@ public:
DummyTransform(llvm::StringRef Name, const TransformOptions &Options)
: Transform(Name, Options) {}
- virtual int apply(const FileOverrides &,
+ virtual int apply(FileOverrides &,
const tooling::CompilationDatabase &,
- const std::vector<std::string> &,
- FileOverrides &) { return 0; }
+ const std::vector<std::string> &) { return 0; }
void setAcceptedChanges(unsigned Changes) {
Transform::setAcceptedChanges(Changes);
@@ -29,7 +28,7 @@ public:
Transform::setDeferredChanges(Changes);
}
- void setOverrides(const FileOverrides &Overrides) {
+ void setOverrides(FileOverrides &Overrides) {
Transform::setOverrides(Overrides);
}
OpenPOWER on IntegriCloud