[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