diff options
| author | Richard Smith <richard-llvm@metafoo.co.uk> | 2012-05-03 17:56:49 +0000 |
|---|---|---|
| committer | Richard Smith <richard-llvm@metafoo.co.uk> | 2012-05-03 17:56:49 +0000 |
| commit | d39d23e610c2a7815515d60c5a538d65d05e8bdc (patch) | |
| tree | 0640205bca4d762bb457dfe247572f0211be2f32 | |
| parent | 3144749f8bf9bbf7c027f2161a930bff80ad6f72 (diff) | |
Fix a couple of cases of (innocuous) unmarked fallthrough. At least one of these
was unintentional. Found by -Wimplicit-fallthrough, patch by Alexander Kornienko!
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@156082 91177308-0d34-0410-b5e6-96231b3b80d8
| -rw-r--r-- | include/clang/AST/RecursiveASTVisitor.h | 1 | ||||
| -rw-r--r-- | include/clang/Analysis/Analyses/FormatString.h | 1 |
2 files changed, 2 insertions, 0 deletions
diff --git a/include/clang/AST/RecursiveASTVisitor.h b/include/clang/AST/RecursiveASTVisitor.h index 8a6e85f226..a9a98d77ba 100644 --- a/include/clang/AST/RecursiveASTVisitor.h +++ b/include/clang/AST/RecursiveASTVisitor.h @@ -1400,6 +1400,7 @@ bool RecursiveASTVisitor<Derived>::TraverseClassInstantiations( case TSK_Undeclared: case TSK_ImplicitInstantiation: TRY_TO(TraverseDecl(SD)); + break; // We don't need to do anything on an explicit instantiation // or explicit specialization because there will be an explicit diff --git a/include/clang/Analysis/Analyses/FormatString.h b/include/clang/Analysis/Analyses/FormatString.h index 9ec27ce91d..f99b97a189 100644 --- a/include/clang/Analysis/Analyses/FormatString.h +++ b/include/clang/Analysis/Analyses/FormatString.h @@ -175,6 +175,7 @@ public: switch (kind) { case PrintErrno: assert(IsPrintf); + return false; case PercentArg: return false; default: |