[clang-tools-extra] r222027 - Remove some redundant virtual on overridden functions

David Blaikie dblaikie at gmail.com
Fri Nov 14 11:27:22 PST 2014


Author: dblaikie
Date: Fri Nov 14 13:27:22 2014
New Revision: 222027

URL: http://llvm.org/viewvc/llvm-project?rev=222027&view=rev
Log:
Remove some redundant virtual on overridden functions

Modified:
    clang-tools-extra/trunk/clang-modernize/Core/IncludeDirectives.cpp
    clang-tools-extra/trunk/clang-modernize/Core/Transform.cpp

Modified: clang-tools-extra/trunk/clang-modernize/Core/IncludeDirectives.cpp
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clang-modernize/Core/IncludeDirectives.cpp?rev=222027&r1=222026&r2=222027&view=diff
==============================================================================
--- clang-tools-extra/trunk/clang-modernize/Core/IncludeDirectives.cpp (original)
+++ clang-tools-extra/trunk/clang-modernize/Core/IncludeDirectives.cpp Fri Nov 14 13:27:22 2014
@@ -178,7 +178,7 @@ private:
     }
   }
 
-  virtual void Endif(SourceLocation Loc, SourceLocation IfLoc) override {
+  void Endif(SourceLocation Loc, SourceLocation IfLoc) override {
     Guard->Count++;
 
     // If it's the #endif corresponding to the top-most #ifndef
@@ -220,7 +220,7 @@ private:
                      const MacroDirective *) override {
     Guard->Count++;
   }
-  virtual void Else(SourceLocation, SourceLocation) override {
+  void Else(SourceLocation, SourceLocation) override {
     Guard->Count++;
   }
 

Modified: clang-tools-extra/trunk/clang-modernize/Core/Transform.cpp
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clang-modernize/Core/Transform.cpp?rev=222027&r1=222026&r2=222027&view=diff
==============================================================================
--- clang-tools-extra/trunk/clang-modernize/Core/Transform.cpp (original)
+++ clang-tools-extra/trunk/clang-modernize/Core/Transform.cpp Fri Nov 14 13:27:22 2014
@@ -39,7 +39,7 @@ public:
   ActionFactory(MatchFinder &Finder, Transform &Owner)
   : Finder(Finder), Owner(Owner) {}
 
-  virtual FrontendAction *create() override {
+  FrontendAction *create() override {
     return new FactoryAdaptor(Finder, Owner);
   }
 
@@ -62,7 +62,7 @@ private:
       return Owner.handleBeginSource(CI, Filename);
     }
 
-    virtual void EndSourceFileAction() override {
+    void EndSourceFileAction() override {
       Owner.handleEndSource();
       return ASTFrontendAction::EndSourceFileAction();
     }





More information about the cfe-commits mailing list