diff options
| author | Alexander Kornienko <alexfh@google.com> | 2015-12-28 13:06:58 +0000 |
|---|---|---|
| committer | Alexander Kornienko <alexfh@google.com> | 2015-12-28 13:06:58 +0000 |
| commit | 9c10490efe30ed75d52810c26e83189a96c04926 (patch) | |
| tree | 6abcb02d4f5e6aea72eca2429f4c5bf290a8ffc9 /clang/lib/StaticAnalyzer/Checkers/ObjCContainersASTChecker.cpp | |
| parent | fba562004bf8a3c383453791bccbac3aaef7bcf0 (diff) | |
| download | bcm5719-llvm-9c10490efe30ed75d52810c26e83189a96c04926.tar.gz bcm5719-llvm-9c10490efe30ed75d52810c26e83189a96c04926.zip | |
Refactor: Simplify boolean conditional return statements in lib/StaticAnalyzer/Checkers
Summary: Use clang-tidy to simplify boolean conditional return values
Reviewers: dcoughlin, krememek
Subscribers: krememek, cfe-commits
Patch by Richard Thomson!
Differential Revision: http://reviews.llvm.org/D10021
llvm-svn: 256491
Diffstat (limited to 'clang/lib/StaticAnalyzer/Checkers/ObjCContainersASTChecker.cpp')
| -rw-r--r-- | clang/lib/StaticAnalyzer/Checkers/ObjCContainersASTChecker.cpp | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/clang/lib/StaticAnalyzer/Checkers/ObjCContainersASTChecker.cpp b/clang/lib/StaticAnalyzer/Checkers/ObjCContainersASTChecker.cpp index 224251beb09..b10ec848ee4 100644 --- a/clang/lib/StaticAnalyzer/Checkers/ObjCContainersASTChecker.cpp +++ b/clang/lib/StaticAnalyzer/Checkers/ObjCContainersASTChecker.cpp @@ -66,9 +66,8 @@ class WalkAST : public StmtVisitor<WalkAST> { // The type must be an array/pointer type. // This could be a null constant, which is allowed. - if (E->isNullPointerConstant(ASTC, Expr::NPC_ValueDependentIsNull)) - return true; - return false; + return static_cast<bool>( + E->isNullPointerConstant(ASTC, Expr::NPC_ValueDependentIsNull)); } public: |
