diff options
author | Edwin Vane <edwin.vane@intel.com> | 2013-06-18 15:31:01 +0000 |
---|---|---|
committer | Edwin Vane <edwin.vane@intel.com> | 2013-06-18 15:31:01 +0000 |
commit | 62c013db6ce8dabac58a313c5e4ca05e75e45881 (patch) | |
tree | bb79b5905ac9a479c6f8b9b7c38bdb3a859fcfc3 /clang-tools-extra/unittests/cpp11-migrate/TransformTest.cpp | |
parent | 302c0da6f138d1475b497a6e987787e94f15046f (diff) | |
download | bcm5719-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.cpp | 7 |
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); } |