[PATCH] D16566: [Clang-tidy] Fix Clang-tidy modernize-use-override warning in unittests/clang-tidy/IncludeInserterTest.cpp; other minor fixes

Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 26 14:36:45 PST 2016


This revision was automatically updated to reflect the committed changes.
Closed by commit rL258870: Fix Clang-tidy modernize-use-override warning in unittests/clang… (authored by eugenezelenko).

Changed prior to commit:
  http://reviews.llvm.org/D16566?vs=45938&id=46054#toc

Repository:
  rL LLVM

http://reviews.llvm.org/D16566

Files:
  clang-tools-extra/trunk/unittests/clang-tidy/IncludeInserterTest.cpp

Index: clang-tools-extra/trunk/unittests/clang-tidy/IncludeInserterTest.cpp
===================================================================
--- clang-tools-extra/trunk/unittests/clang-tidy/IncludeInserterTest.cpp
+++ clang-tools-extra/trunk/unittests/clang-tidy/IncludeInserterTest.cpp
@@ -98,7 +98,6 @@
 public:
   CXXSystemIncludeInserterCheck(StringRef CheckName, ClangTidyContext *Context)
       : IncludeInserterCheckBase(CheckName, Context) {}
-  virtual ~CXXSystemIncludeInserterCheck() {}
 
   std::vector<StringRef> HeadersToInclude() const override { return {"set"}; }
   bool IsAngledInclude() const override { return true; }
@@ -522,7 +521,7 @@
                                    "insert_includes_test_header.cc"));
 }
 
-} // namespace
+} // anonymous namespace
 } // namespace tidy
 } // namespace clang
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D16566.46054.patch
Type: text/x-patch
Size: 826 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160126/de38115e/attachment.bin>


More information about the cfe-commits mailing list