diff options
3 files changed, 7 insertions, 5 deletions
diff --git a/clang/lib/StaticAnalyzer/Checkers/UninitializedObject/UninitializedObject.h b/clang/lib/StaticAnalyzer/Checkers/UninitializedObject/UninitializedObject.h index 07751a92360..ee9bcecb4ef 100644 --- a/clang/lib/StaticAnalyzer/Checkers/UninitializedObject/UninitializedObject.h +++ b/clang/lib/StaticAnalyzer/Checkers/UninitializedObject/UninitializedObject.h @@ -32,6 +32,8 @@ class FieldNode { protected: const FieldRegion *FR; + ~FieldNode() = default; + public: FieldNode(const FieldRegion *FR) : FR(FR) { assert(FR); } diff --git a/clang/lib/StaticAnalyzer/Checkers/UninitializedObject/UninitializedObjectChecker.cpp b/clang/lib/StaticAnalyzer/Checkers/UninitializedObject/UninitializedObjectChecker.cpp index 876aebb477c..4da5fa5461b 100644 --- a/clang/lib/StaticAnalyzer/Checkers/UninitializedObject/UninitializedObjectChecker.cpp +++ b/clang/lib/StaticAnalyzer/Checkers/UninitializedObject/UninitializedObjectChecker.cpp @@ -46,7 +46,7 @@ // //===----------------------------------------------------------------------===// -#include "ClangSACheckers.h" +#include "../ClangSACheckers.h" #include "UninitializedObject.h" #include "clang/StaticAnalyzer/Core/BugReporter/BugType.h" #include "clang/StaticAnalyzer/Core/Checker.h" @@ -74,7 +74,7 @@ public: /// A basic field type, that is not a pointer or a reference, it's dynamic and /// static type is the same. -class RegularField : public FieldNode { +class RegularField final : public FieldNode { public: RegularField(const FieldRegion *FR) : FieldNode(FR) {} @@ -84,7 +84,7 @@ public: virtual void printPrefix(llvm::raw_ostream &Out) const override {} - virtual void printNode(llvm::raw_ostream &Out) const { + virtual void printNode(llvm::raw_ostream &Out) const override { Out << getVariableName(getDecl()); } diff --git a/clang/lib/StaticAnalyzer/Checkers/UninitializedObject/UninitializedPointee.cpp b/clang/lib/StaticAnalyzer/Checkers/UninitializedObject/UninitializedPointee.cpp index 61e96ef9ae7..7f08d028942 100644 --- a/clang/lib/StaticAnalyzer/Checkers/UninitializedObject/UninitializedPointee.cpp +++ b/clang/lib/StaticAnalyzer/Checkers/UninitializedObject/UninitializedPointee.cpp @@ -18,7 +18,7 @@ // //===----------------------------------------------------------------------===// -#include "ClangSACheckers.h" +#include "../ClangSACheckers.h" #include "UninitializedObject.h" #include "clang/StaticAnalyzer/Core/BugReporter/BugType.h" #include "clang/StaticAnalyzer/Core/Checker.h" @@ -31,7 +31,7 @@ using namespace clang::ento; namespace { /// Represents a pointer or a reference field. -class LocField : public FieldNode { +class LocField final : public FieldNode { /// We'll store whether the pointee or the pointer itself is uninitialited. const bool IsDereferenced; |
