From 94425336fd29c9715aec10318e30b18ac8ba75ea Mon Sep 17 00:00:00 2001
From: Christian Kandeler <christian.kandeler@qt.io>
Date: Tue, 12 Feb 2019 13:11:15 +0100
Subject: [PATCH] ClangTools: Fix fall-through warning

GCC does not realize that all cases lead to a return statement.

Change-Id: I0b6fc7b349b8dc09b739f3423121d2e760c74701
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@qt.io>
---
 src/plugins/clangtools/clangtoolsdiagnosticmodel.cpp | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/plugins/clangtools/clangtoolsdiagnosticmodel.cpp b/src/plugins/clangtools/clangtoolsdiagnosticmodel.cpp
index 85a9ade878b..4225970221d 100644
--- a/src/plugins/clangtools/clangtoolsdiagnosticmodel.cpp
+++ b/src/plugins/clangtools/clangtoolsdiagnosticmodel.cpp
@@ -428,6 +428,7 @@ QVariant DiagnosticItem::data(int column, int role) const
             case FixitStatus::NotScheduled:
                 return true;
             }
+            break;
         case Qt::CheckStateRole: {
             switch (m_fixitStatus) {
             case FixitStatus::NotAvailable:
@@ -439,6 +440,7 @@ QVariant DiagnosticItem::data(int column, int role) const
             case FixitStatus::Scheduled:
                 return Qt::Checked;
             }
+            break;
         }
         case Qt::DisplayRole:
             return QString("%1: %2").arg(lineColumnString(m_diagnostic.location),