[PATCH] D24178: [clang-rename] Fix some Clang-tidy modernize-use-override and Include What You Use warnings; other minor fixes

Kirill Bobyrev via cfe-commits cfe-commits at lists.llvm.org
Sun Sep 4 15:28:16 PDT 2016


This revision was automatically updated to reflect the committed changes.
Closed by commit rL280638: [clang-rename] Fix Clang-tidy and IWYU warnings; other minor fixes (authored by omtcyfz).

Changed prior to commit:
  https://reviews.llvm.org/D24178?vs=70119&id=70293#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D24178

Files:
  clang-tools-extra/trunk/clang-rename/USRFinder.h

Index: clang-tools-extra/trunk/clang-rename/USRFinder.h
===================================================================
--- clang-tools-extra/trunk/clang-rename/USRFinder.h
+++ clang-tools-extra/trunk/clang-rename/USRFinder.h
@@ -12,18 +12,21 @@
 /// code.
 ///
 //===----------------------------------------------------------------------===//
+
 #ifndef LLVM_CLANG_TOOLS_EXTRA_CLANG_RENAME_USR_FINDER_H
 #define LLVM_CLANG_TOOLS_EXTRA_CLANG_RENAME_USR_FINDER_H
 
 #include "clang/AST/AST.h"
 #include "clang/AST/ASTContext.h"
 #include "clang/ASTMatchers/ASTMatchFinder.h"
 #include <string>
+#include <vector>
 
 using namespace llvm;
 using namespace clang::ast_matchers;
 
 namespace clang {
+
 class ASTContext;
 class Decl;
 class SourceLocation;
@@ -64,7 +67,7 @@
     Finder.addMatcher(NestedNameSpecifierLocMatcher, this);
   }
 
-  virtual void run(const MatchFinder::MatchResult &Result) {
+  void run(const MatchFinder::MatchResult &Result) override {
     const auto *NNS = Result.Nodes.getNodeAs<NestedNameSpecifierLoc>(
         "nestedNameSpecifierLoc");
     Locations.push_back(*NNS);
@@ -74,7 +77,8 @@
   std::vector<NestedNameSpecifierLoc> Locations;
   MatchFinder Finder;
 };
-}
-}
+
+} // namespace rename
+} // namespace clang
 
 #endif // LLVM_CLANG_TOOLS_EXTRA_CLANG_RENAME_USR_FINDER_H


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D24178.70293.patch
Type: text/x-patch
Size: 1317 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160904/d34a3af5/attachment.bin>


More information about the cfe-commits mailing list