[llvm-branch-commits] [cfe-branch] r155118 - in /cfe/branches/tooling: include/clang/Tooling/Refactoring.h unittests/Tooling/RefactoringTest.cpp

[Manuel Klimek]

Author: klimek
Date: Thu Apr 19 06:21:17 2012
New Revision: 155118

URL: http://llvm.org/viewvc/llvm-project?rev=155118&view=rev
Log:
Minor clean ups to the refactoring tools.

Modified:
    cfe/branches/tooling/include/clang/Tooling/Refactoring.h
    cfe/branches/tooling/unittests/Tooling/RefactoringTest.cpp

Modified: cfe/branches/tooling/include/clang/Tooling/Refactoring.h
URL: http://llvm.org/viewvc/llvm-project/cfe/branches/tooling/include/clang/Tooling/Refactoring.h?rev=155118&r1=155117&r2=155118&view=diff
==============================================================================
--- cfe/branches/tooling/include/clang/Tooling/Refactoring.h (original)
+++ cfe/branches/tooling/include/clang/Tooling/Refactoring.h Thu Apr 19 06:21:17 2012
@@ -101,8 +101,8 @@
 /// \brief Apply all replacements on the Rewriter.
 ///
 /// If at least one Apply returns false, ApplyAll returns false. Every
-/// Apply will be executed independently of the result of the result of
-/// other Apply operations.
+/// Apply will be executed independently of the result of other
+/// Apply operations.
 bool applyAllReplacements(Replacements &Replaces, Rewriter &Rewrite);
 
 /// \brief Saves all changed files in the Rewriter to disk.
@@ -114,10 +114,9 @@
 /// \brief A tool to run refactorings.
 ///
 /// This is a refactoring specific version of \see ClangTool.
-/// All text replacements added to GetReplacements() during the run of the
+/// All text replacements added to getReplacements() during the run of the
 /// tool will be applied and saved after all translation units have been
 /// processed.
-/// FIXME: Figure out the correct relationship to ClangTool.
 class RefactoringTool {
 public:
   /// \see ClangTool::ClangTool.
@@ -129,7 +128,7 @@
   /// processed.
   Replacements &getReplacements();
 
-  /// \see ClangTool::Run.
+  /// \see ClangTool::run.
   int run(FrontendActionFactory *ActionFactory);
 
 private:

Modified: cfe/branches/tooling/unittests/Tooling/RefactoringTest.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/branches/tooling/unittests/Tooling/RefactoringTest.cpp?rev=155118&r1=155117&r2=155118&view=diff
==============================================================================
--- cfe/branches/tooling/unittests/Tooling/RefactoringTest.cpp (original)
+++ cfe/branches/tooling/unittests/Tooling/RefactoringTest.cpp Thu Apr 19 06:21:17 2012
@@ -24,7 +24,6 @@
 #include "clang/Tooling/Tooling.h"
 #include "llvm/ADT/SmallString.h"
 #include "llvm/Support/Path.h"
-#include "llvm/Support/raw_os_ostream.h"
 #include "gtest/gtest.h"
 
 namespace clang {
@@ -322,7 +321,6 @@
 class CallToFVisitor : public TestVisitor<CallToFVisitor> {
 public:
   bool VisitCallExpr(CallExpr *Call) {
-    llvm::errs() << Call->getDirectCallee()->getName() << "\n";
     if (Call->getDirectCallee()->getName() == "F") {
       Replace = Replacement(*SM, Call, "");
     }