Fixed some renaming detritus in the comments, and an unused variable warning. Thanks to David Blaikie and Manuel Klimek for the comments

llvm-svn: 162031
This commit is contained in:
Marshall Clow 2012-08-16 17:47:06 +00:00
parent 9dcf671d13
commit 22626c3d7e

View File

@ -1,4 +1,4 @@
//===- tools/extra/RefactoringTemplate.cpp - Template for refactoring tool ===// //===- tools/extra/ToolTemplate.cpp - Template for refactoring tool ===//
// //
// The LLVM Compiler Infrastructure // The LLVM Compiler Infrastructure
// //
@ -11,7 +11,7 @@
// The goal is to lower the "barrier to entry" for writing refactoring tools. // The goal is to lower the "barrier to entry" for writing refactoring tools.
// //
// Usage: // Usage:
// refactoringTemplate <cmake-output-dir> <file1> <file2> ... // tool-template <cmake-output-dir> <file1> <file2> ...
// //
// Where <cmake-output-dir> is a CMake build directory in which a file named // Where <cmake-output-dir> is a CMake build directory in which a file named
// compile_commands.json exists (enable -DCMAKE_EXPORT_COMPILE_COMMANDS in // compile_commands.json exists (enable -DCMAKE_EXPORT_COMPILE_COMMANDS in
@ -26,11 +26,11 @@
// removed, but the rest of a relative path must be a suffix of a path in // removed, but the rest of a relative path must be a suffix of a path in
// the compile command line database. // the compile command line database.
// //
// For example, to use refactoringTemplate on all files in a subtree of the // For example, to use tool-template on all files in a subtree of the
// source tree, use: // source tree, use:
// //
// /path/in/subtree $ find . -name '*.cpp'| // /path/in/subtree $ find . -name '*.cpp'|
// xargs refactoringTemplate /path/to/build // xargs tool-template /path/to/build
// //
//===----------------------------------------------------------------------===// //===----------------------------------------------------------------------===//
@ -52,14 +52,15 @@ using namespace clang::tooling;
using namespace llvm; using namespace llvm;
namespace { namespace {
class TemplateCallback : public MatchFinder::MatchCallback { class ToolTemplateCallback : public MatchFinder::MatchCallback {
public: public:
TemplateCallback(Replacements *Replace) : Replace(Replace) {} ToolTemplateCallback(Replacements *Replace) : Replace(Replace) {}
virtual void run(const MatchFinder::MatchResult &Result) { virtual void run(const MatchFinder::MatchResult &Result) {
// TODO: This routine will get called for each thing that the matchers find. // TODO: This routine will get called for each thing that the matchers find.
// At this point, you can examine the match, and do whatever you want, // At this point, you can examine the match, and do whatever you want,
// including replacing the matched text with other text // including replacing the matched text with other text
(void) Replace; // This to prevent an "unused member variable" warning;
} }
private: private:
@ -95,7 +96,7 @@ int main(int argc, const char **argv) {
} }
RefactoringTool Tool(*Compilations, SourcePaths); RefactoringTool Tool(*Compilations, SourcePaths);
ast_matchers::MatchFinder Finder; ast_matchers::MatchFinder Finder;
TemplateCallback Callback(&Tool.getReplacements()); ToolTemplateCallback Callback(&Tool.getReplacements());
// TODO: Put your matchers here. // TODO: Put your matchers here.
// Use Finder.addMatcher(...) to define the patterns in the AST that you // Use Finder.addMatcher(...) to define the patterns in the AST that you