Support C++17 fallthrough attribute

Replaces our mix of comments for annotating intended absence of break
in switches with the C++17 attribute [[fallthrough]], or its earlier
a clang extension counterpart.

Change-Id: I4b2d0b9b5e4425819c7f1bf01608093c536b6d14
Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>

1 files changed, 4 insertions, 4 deletions

diff --git a/src/tools/qdbuscpp2xml/qdbuscpp2xml.cpp b/src/tools/qdbuscpp2xml/qdbuscpp2xml.cpp
index 083187555d..26e7b086d9 100644
--- a/src/tools/qdbuscpp2xml/qdbuscpp2xml.cpp
+++ b/src/tools/qdbuscpp2xml/qdbuscpp2xml.cpp
@@ -335,28 +335,28 @@ static void parseCmdLine(QStringList &arguments)
         switch (c) {
         case 'P':
             flags |= QDBusConnection::ExportNonScriptableProperties;
-            // fall through
+            Q_FALLTHROUGH();
         case 'p':
             flags |= QDBusConnection::ExportScriptableProperties;
             break;
 
         case 'S':
             flags |= QDBusConnection::ExportNonScriptableSignals;
-            // fall through
+            Q_FALLTHROUGH();
         case 's':
             flags |= QDBusConnection::ExportScriptableSignals;
             break;
 
         case 'M':
             flags |= QDBusConnection::ExportNonScriptableSlots;
-            // fall through
+            Q_FALLTHROUGH();
         case 'm':
             flags |= QDBusConnection::ExportScriptableSlots;
             break;
 
         case 'A':
             flags |= QDBusConnection::ExportNonScriptableContents;
-            // fall through
+            Q_FALLTHROUGH();
         case 'a':
             flags |= QDBusConnection::ExportScriptableContents;
             break;