Merge pull request #1038 from marxin/fix-wimplicit-fallthrough

Add [[gnu::fallthrough]] to cases which fall through.
Andersbakken · Aug 30, 2017 · 320317d · 320317d
2 parents 93b0063 + 0c0942e
commit 320317d
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 0 deletions.
diff --git a/src/ClangIndexer.cpp b/src/ClangIndexer.cpp
@@ -473,6 +473,7 @@ String ClangIndexer::addNamePermutations(const CXCursor &cursor, Location locati
                 // namespaces can include all namespaces in their symbolname
                 if (originalKind == CXCursor_Namespace)
                     break;
+                [[gnu::fallthrough]];
             default:
                 cutoff = pos;
                 break;

diff --git a/src/IncludeFileJob.cpp b/src/IncludeFileJob.cpp
@@ -76,6 +76,7 @@ int IncludeFileJob::execute()
                 case CXCursor_TypedefDecl:
                     if (!sym.isDefinition())
                         break;
+                    [[gnu::fallthrough]];
                 case CXCursor_FunctionDecl:
                 case CXCursor_FunctionTemplate: {
                     List<String> alternatives;

diff --git a/src/Project.cpp b/src/Project.cpp
@@ -1559,6 +1559,7 @@ Set<Symbol> Project::findTargets(const Symbol &symbol)
         case CXCursor_StructDecl:
             if (symbol.isDefinition() && !(symbol.flags & Symbol::TemplateSpecialization))
                 return ret;
+            [[gnu::fallthrough]];
         case CXCursor_FunctionDecl:
         case CXCursor_CXXMethod:
         case CXCursor_Destructor: