Merge pull request #7530 from m-kuhn/implicit-fallthrough

m-kuhn · web-flow · commit 3fc8e8d61b95 · 2018-08-06T16:43:43.000+02:00
Enable -Wimplicit-fallthrough
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -461,7 +461,7 @@ IF (PEDANTIC)
     # ADD_DEFINITIONS( -fstrict-aliasing -Wstrict-aliasing=1 -Wredundant-decls )
 
     IF ("${CMAKE_CXX_COMPILER_ID}" MATCHES "Clang")
-      SET(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-return-type-c-linkage -Wno-overloaded-virtual")
+      SET(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-return-type-c-linkage -Wno-overloaded-virtual -Wimplicit-fallthrough")
     ENDIF ("${CMAKE_CXX_COMPILER_ID}" MATCHES "Clang")
 
     # add any extra CXXFLAGS flags set by user. can be -D CXX_EXTRA_FLAGS or environment variable
diff --git a/external/libdxfrw/intern/dwgutil.cpp b/external/libdxfrw/intern/dwgutil.cpp
@@ -19,7 +19,7 @@
 #if __cplusplus >= 201500
 #define FALLTHROUGH [[fallthrough]];
 #elif defined(__clang__)
-#define FALLTHROUGH //[[clang::fallthrough]]
+#define FALLTHROUGH [[clang::fallthrough]];
 #elif defined(__GNUC__) && __GNUC__ >= 7
 #define FALLTHROUGH [[gnu::fallthrough]];
 #else
@@ -28,7 +28,8 @@
 
 #include "qgslogger.h"
 
-/** Utility function
+/**
+ * Utility function
  * convert a int to string in hex
  **/
 namespace DRW
diff --git a/external/libdxfrw/libdxfrw.cpp b/external/libdxfrw/libdxfrw.cpp
@@ -27,7 +27,7 @@
 #if __cplusplus >= 201500
 #define FALLTHROUGH [[fallthrough]];
 #elif defined(__clang__)
-#define FALLTHROUGH //[[clang::fallthrough]]
+#define FALLTHROUGH [[clang::fallthrough]];
 #elif defined(__GNUC__) && __GNUC__ >= 7
 #define FALLTHROUGH [[gnu::fallthrough]];
 #else
diff --git a/src/core/dxf/qgsdxfexport.cpp b/src/core/dxf/qgsdxfexport.cpp
@@ -3782,7 +3782,7 @@ void QgsDxfExport::addFeature( QgsSymbolRenderContext &ctx, const QgsCoordinateT
         tempGeom = geom->segmentize();
         if ( !tempGeom )
           break;
-        FALLTHROUGH;
+        FALLTHROUGH
       case QgsWkbTypes::LineString:
         if ( !qgsDoubleNear( offset, 0.0 ) )
         {
@@ -3802,7 +3802,7 @@ void QgsDxfExport::addFeature( QgsSymbolRenderContext &ctx, const QgsCoordinateT
         tempGeom = geom->segmentize();
         if ( !tempGeom )
           break;
-        FALLTHROUGH;
+        FALLTHROUGH
       case QgsWkbTypes::MultiLineString:
       {
         if ( !qgsDoubleNear( offset, 0.0 ) )
@@ -3828,7 +3828,7 @@ void QgsDxfExport::addFeature( QgsSymbolRenderContext &ctx, const QgsCoordinateT
         tempGeom = geom->segmentize();
         if ( !tempGeom )
           break;
-        FALLTHROUGH;
+        FALLTHROUGH
       case QgsWkbTypes::Polygon:
       {
         if ( !qgsDoubleNear( offset, 0.0 ) )
@@ -3888,7 +3888,7 @@ void QgsDxfExport::addFeature( QgsSymbolRenderContext &ctx, const QgsCoordinateT
         tempGeom = tempGeom->segmentize();
         if ( !tempGeom )
           break;
-        FALLTHROUGH;
+        FALLTHROUGH
       case QgsWkbTypes::Polygon:
         writePolygon( tempGeom->coordinateSequence().at( 0 ), layer, QStringLiteral( "SOLID" ), brushColor );
         break;
diff --git a/src/core/expression/qgsexpressionnodeimpl.cpp b/src/core/expression/qgsexpressionnodeimpl.cpp
@@ -193,7 +193,7 @@ QVariant QgsExpressionNodeBinaryOperator::evalNode( QgsExpression *parent, const
         return QVariant( sL + sR );
       }
       //intentional fall-through
-      FALLTHROUGH;
+      FALLTHROUGH
     case boMinus:
     case boMul:
     case boDiv:
diff --git a/src/core/layout/qgsabstractreportsection.cpp b/src/core/layout/qgsabstractreportsection.cpp
@@ -221,7 +221,7 @@ bool QgsAbstractReportSection::next()
 
       // but if not, then the current section is a body
       mNextSection = Body;
-      FALLTHROUGH;
+      FALLTHROUGH
     }
 
     case Body:
@@ -238,7 +238,7 @@ bool QgsAbstractReportSection::next()
         return true;
       }
 
-      FALLTHROUGH;
+      FALLTHROUGH
     }
 
     case Children:
@@ -284,7 +284,7 @@ bool QgsAbstractReportSection::next()
 
       // all children and bodies have spent their content, so move to the footer
       mNextSection = Footer;
-      FALLTHROUGH;
+      FALLTHROUGH
     }
 
     case Footer:
@@ -303,7 +303,7 @@ bool QgsAbstractReportSection::next()
       }
 
       // if not, then we're all done
-      FALLTHROUGH;
+      FALLTHROUGH
     }
 
     case End:
diff --git a/src/core/qgis.h b/src/core/qgis.h
@@ -563,7 +563,7 @@ typedef unsigned long long qgssize;
 #if __cplusplus >= 201500
 #define FALLTHROUGH [[fallthrough]];
 #elif defined(__clang__)
-#define FALLTHROUGH //[[clang::fallthrough]]
+#define FALLTHROUGH [[clang::fallthrough]];
 #elif defined(__GNUC__) && __GNUC__ >= 7
 #define FALLTHROUGH [[gnu::fallthrough]];
 #else
diff --git a/src/core/qgsogcutils.cpp b/src/core/qgsogcutils.cpp
@@ -1212,7 +1212,7 @@ QDomElement QgsOgcUtils::geometryToGML( const QgsGeometry &geometry, QDomDocumen
       case QgsWkbTypes::MultiPoint25D:
         hasZValue = true;
         //intentional fall-through
-        FALLTHROUGH;
+        FALLTHROUGH
       case QgsWkbTypes::MultiPoint:
       {
         QDomElement multiPointElem = doc.createElement( QStringLiteral( "gml:MultiPoint" ) );
@@ -1256,7 +1256,7 @@ QDomElement QgsOgcUtils::geometryToGML( const QgsGeometry &geometry, QDomDocumen
       case QgsWkbTypes::LineString25D:
         hasZValue = true;
         //intentional fall-through
-        FALLTHROUGH;
+        FALLTHROUGH
       case QgsWkbTypes::LineString:
       {
         QDomElement lineStringElem = doc.createElement( QStringLiteral( "gml:LineString" ) );
@@ -1298,7 +1298,7 @@ QDomElement QgsOgcUtils::geometryToGML( const QgsGeometry &geometry, QDomDocumen
       case QgsWkbTypes::MultiLineString25D:
         hasZValue = true;
         //intentional fall-through
-        FALLTHROUGH;
+        FALLTHROUGH
       case QgsWkbTypes::MultiLineString:
       {
         QDomElement multiLineStringElem = doc.createElement( QStringLiteral( "gml:MultiLineString" ) );
@@ -1355,7 +1355,7 @@ QDomElement QgsOgcUtils::geometryToGML( const QgsGeometry &geometry, QDomDocumen
       case QgsWkbTypes::Polygon25D:
         hasZValue = true;
         //intentional fall-through
-        FALLTHROUGH;
+        FALLTHROUGH
       case QgsWkbTypes::Polygon:
       {
         QDomElement polygonElem = doc.createElement( QStringLiteral( "gml:Polygon" ) );
@@ -1420,7 +1420,7 @@ QDomElement QgsOgcUtils::geometryToGML( const QgsGeometry &geometry, QDomDocumen
       case QgsWkbTypes::MultiPolygon25D:
         hasZValue = true;
         //intentional fall-through
-        FALLTHROUGH;
+        FALLTHROUGH
       case QgsWkbTypes::MultiPolygon:
       {
         QDomElement multiPolygonElem = doc.createElement( QStringLiteral( "gml:MultiPolygon" ) );
diff --git a/src/core/qgspointlocator.cpp b/src/core/qgspointlocator.cpp
@@ -357,7 +357,7 @@ static QgsPointLocator::MatchList _geometrySegmentsInRect( QgsGeometry *geom, co
     case QgsWkbTypes::LineString25D:
       hasZValue = true;
       //intentional fall-through
-      FALLTHROUGH;
+      FALLTHROUGH
     case QgsWkbTypes::LineString:
     {
       int nPoints;
@@ -391,7 +391,7 @@ static QgsPointLocator::MatchList _geometrySegmentsInRect( QgsGeometry *geom, co
     case QgsWkbTypes::MultiLineString25D:
       hasZValue = true;
       //intentional fall-through
-      FALLTHROUGH;
+      FALLTHROUGH
     case QgsWkbTypes::MultiLineString:
     {
       int nLines;
@@ -432,7 +432,7 @@ static QgsPointLocator::MatchList _geometrySegmentsInRect( QgsGeometry *geom, co
     case QgsWkbTypes::Polygon25D:
       hasZValue = true;
       //intentional fall-through
-      FALLTHROUGH;
+      FALLTHROUGH
     case QgsWkbTypes::Polygon:
     {
       int nRings;
@@ -473,7 +473,7 @@ static QgsPointLocator::MatchList _geometrySegmentsInRect( QgsGeometry *geom, co
     case QgsWkbTypes::MultiPolygon25D:
       hasZValue = true;
       //intentional fall-through
-      FALLTHROUGH;
+      FALLTHROUGH
     case QgsWkbTypes::MultiPolygon:
     {
       int nPolygons;
diff --git a/src/core/qgsrendercontext.cpp b/src/core/qgsrendercontext.cpp
@@ -273,7 +273,7 @@ double QgsRenderContext::convertToPainterUnits( double size, QgsUnitTypes::Rende
       size = convertMetersToMapUnits( size );
       unit = QgsUnitTypes::RenderMapUnits;
       // Fall through to RenderMapUnits with size in meters converted to size in MapUnits
-      FALLTHROUGH;
+      FALLTHROUGH
     }
     case QgsUnitTypes::RenderMapUnits:
     {
@@ -323,7 +323,7 @@ double QgsRenderContext::convertToMapUnits( double size, QgsUnitTypes::RenderUni
     {
       size = convertMetersToMapUnits( size );
       // Fall through to RenderMapUnits with values of meters converted to MapUnits
-      FALLTHROUGH;
+      FALLTHROUGH
     }
     case QgsUnitTypes::RenderMapUnits:
     {
diff --git a/src/core/qgstextrenderer.cpp b/src/core/qgstextrenderer.cpp
@@ -1813,7 +1813,7 @@ void QgsTextRenderer::drawPart( const QRectF &rect, double rotation, HAlignment
       if ( !format.buffer().enabled() )
         break;
     }
-    FALLTHROUGH;
+    FALLTHROUGH
     case Text:
     case Shadow:
     {
@@ -1857,7 +1857,7 @@ void QgsTextRenderer::drawPart( QPointF origin, double rotation, QgsTextRenderer
       if ( !format.buffer().enabled() )
         break;
     }
-    FALLTHROUGH;
+    FALLTHROUGH
     case Text:
     case Shadow:
     {
@@ -2077,6 +2077,7 @@ void QgsTextRenderer::drawBackground( QgsRenderContext &context, QgsTextRenderer
                                         component.origin.y() - height / 2.0 + originAdjust );
             break;
         }
+        break;
       }
 
       case Label:
diff --git a/src/core/qgsvectorlayer.cpp b/src/core/qgsvectorlayer.cpp
@@ -3464,7 +3464,7 @@ QSet<QVariant> QgsVectorLayer::uniqueValues( int index, int limit ) const
         uniqueValues = mDataProvider->uniqueValues( index, limit );
         return uniqueValues;
       }
-      FALLTHROUGH;
+      FALLTHROUGH
     //we need to go through each feature
     case QgsFields::OriginJoin:
     case QgsFields::OriginExpression:
@@ -3561,7 +3561,7 @@ QStringList QgsVectorLayer::uniqueStringsMatching( int index, const QString &sub
       {
         return mDataProvider->uniqueStringsMatching( index, substring, limit, feedback );
       }
-      FALLTHROUGH;
+      FALLTHROUGH
     //we need to go through each feature
     case QgsFields::OriginJoin:
     case QgsFields::OriginExpression:
@@ -3654,7 +3654,7 @@ QVariant QgsVectorLayer::minimumValue( int index ) const
         return mDataProvider->minimumValue( index );
       }
     }
-    FALLTHROUGH;
+    FALLTHROUGH
     // no choice but to go through all features
     case QgsFields::OriginExpression:
     case QgsFields::OriginJoin:
@@ -3740,7 +3740,7 @@ QVariant QgsVectorLayer::maximumValue( int index ) const
         return mDataProvider->maximumValue( index );
       }
 
-      FALLTHROUGH;
+      FALLTHROUGH
     //no choice but to go through each feature
     case QgsFields::OriginJoin:
     case QgsFields::OriginExpression:
diff --git a/src/core/symbology/qgssymbol.cpp b/src/core/symbology/qgssymbol.cpp
@@ -996,7 +996,7 @@ void QgsSymbol::renderFeature( const QgsFeature &feature, QgsRenderContext &cont
         break;
       }
 
-      FALLTHROUGH;
+      FALLTHROUGH
     }
     default:
       QgsDebugMsg( QString( "feature %1: unsupported wkb type %2/%3 for rendering" )
diff --git a/src/gui/qgspropertyoverridebutton.cpp b/src/gui/qgspropertyoverridebutton.cpp
@@ -119,12 +119,12 @@ void QgsPropertyOverrideButton::init( int propertyKey, const QgsProperty &proper
   {
     case QgsPropertyDefinition::DataTypeBoolean:
       ts << tr( "boolean" );
-      FALLTHROUGH;
+      FALLTHROUGH
 
     case QgsPropertyDefinition::DataTypeNumeric:
       ts << tr( "int" );
       ts << tr( "double" );
-      FALLTHROUGH;
+      FALLTHROUGH
 
     case QgsPropertyDefinition::DataTypeString:
       ts << tr( "string" );
diff --git a/src/gui/symbology/qgssymbolslistwidget.cpp b/src/gui/symbology/qgssymbolslistwidget.cpp
@@ -175,16 +175,20 @@ void QgsSymbolsListWidget::createAuxiliaryField()
       if ( markerSymbol )
         markerSymbol->setDataDefinedAngle( button->toProperty() );
       break;
+
     case QgsSymbolLayer::PropertySize:
       if ( markerSymbol )
       {
         markerSymbol->setDataDefinedSize( button->toProperty() );
         markerSymbol->setScaleMethod( QgsSymbol::ScaleDiameter );
       }
       break;
+
     case QgsSymbolLayer::PropertyStrokeWidth:
       if ( lineSymbol )
         lineSymbol->setDataDefinedWidth( button->toProperty() );
+      break;
+
     default:
       break;
   }
diff --git a/src/plugins/georeferencer/qgsgeorefplugingui.cpp b/src/plugins/georeferencer/qgsgeorefplugingui.cpp
@@ -395,7 +395,7 @@ void QgsGeorefPluginGui::generateGDALScript()
         break;
       }
     }
-    FALLTHROUGH;
+    FALLTHROUGH
     default:
       mMessageBar->pushMessage( tr( "Invalid Transform" ), tr( "GDAL scripting is not supported for %1 transformation." )
                                 .arg( convertTransformEnumToString( mTransformParam ) )
diff --git a/src/plugins/grass/qgsgrassnewmapset.cpp b/src/plugins/grass/qgsgrassnewmapset.cpp
@@ -322,7 +322,7 @@ int QgsGrassNewMapset::nextId() const
         id = MapSet;
         break;
       }
-      FALLTHROUGH;
+      FALLTHROUGH
     case Database:
     case Crs:
     case Region:
diff --git a/src/providers/ogr/qgsogrprovider.cpp b/src/providers/ogr/qgsogrprovider.cpp
@@ -1619,6 +1619,8 @@ bool QgsOgrProvider::addAttributeOGRLevel( const QgsField &field, bool &ignoreEr
   {
     case QVariant::Bool:
       OGR_Fld_SetSubType( fielddefn.get(), OFSTBoolean );
+      break;
+
     default:
       break;
   }
diff --git a/src/providers/oracle/qgsoracleprovider.cpp b/src/providers/oracle/qgsoracleprovider.cpp
@@ -1841,7 +1841,7 @@ void QgsOracleProvider::appendGeomParam( const QgsGeometry &geom, QSqlQuery &qry
       case QgsWkbTypes::Point25D:
       case QgsWkbTypes::PointZ:
         dim = 3;
-        FALLTHROUGH;
+        FALLTHROUGH
 
       case QgsWkbTypes::Point:
         g.srid  = mSrid;
@@ -1856,7 +1856,7 @@ void QgsOracleProvider::appendGeomParam( const QgsGeometry &geom, QSqlQuery &qry
       case QgsWkbTypes::LineStringZ:
       case QgsWkbTypes::MultiLineStringZ:
         dim = 3;
-        FALLTHROUGH;
+        FALLTHROUGH
 
       case QgsWkbTypes::LineString:
       case QgsWkbTypes::MultiLineString:
@@ -1895,7 +1895,7 @@ void QgsOracleProvider::appendGeomParam( const QgsGeometry &geom, QSqlQuery &qry
       case QgsWkbTypes::PolygonZ:
       case QgsWkbTypes::MultiPolygonZ:
         dim = 3;
-        FALLTHROUGH;
+        FALLTHROUGH
 
       case QgsWkbTypes::Polygon:
       case QgsWkbTypes::MultiPolygon:
@@ -1937,7 +1937,7 @@ void QgsOracleProvider::appendGeomParam( const QgsGeometry &geom, QSqlQuery &qry
       case QgsWkbTypes::MultiPoint25D:
       case QgsWkbTypes::MultiPointZ:
         dim = 3;
-        FALLTHROUGH;
+        FALLTHROUGH
 
       case QgsWkbTypes::MultiPoint:
       {
diff --git a/src/providers/postgres/qgspostgresconn.cpp b/src/providers/postgres/qgspostgresconn.cpp
diff --git a/src/providers/postgres/qgspostgresexpressioncompiler.cpp b/src/providers/postgres/qgspostgresexpressioncompiler.cpp
diff --git a/src/providers/spatialite/qgsspatialiteprovider.cpp b/src/providers/spatialite/qgsspatialiteprovider.cpp

Original file line number	Diff line number	Diff line change
`@@ -193,7 +193,7 @@ QVariant QgsExpressionNodeBinaryOperator::evalNode( QgsExpression *parent, const`
`193`	`193`	`return QVariant( sL + sR );`
`194`	`194`	`}`
`195`	`195`	`//intentional fall-through`
`196`		`- FALLTHROUGH;`
	`196`	`+ FALLTHROUGH`
`197`	`197`	`case boMinus:`
`198`	`198`	`case boMul:`
`199`	`199`	`case boDiv:`
Original file line number	Diff line number	Diff line change
`@@ -221,7 +221,7 @@ bool QgsAbstractReportSection::next()`
`221`	`221`
`222`	`222`	`// but if not, then the current section is a body`
`223`	`223`	`mNextSection = Body;`
`224`		`- FALLTHROUGH;`
	`224`	`+ FALLTHROUGH`
`225`	`225`	`}`
`226`	`226`
`227`	`227`	`case Body:`
`@@ -238,7 +238,7 @@ bool QgsAbstractReportSection::next()`
`238`	`238`	`return true;`
`239`	`239`	`}`
`240`	`240`
`241`		`- FALLTHROUGH;`
	`241`	`+ FALLTHROUGH`
`242`	`242`	`}`
`243`	`243`
`244`	`244`	`case Children:`
`@@ -284,7 +284,7 @@ bool QgsAbstractReportSection::next()`
`284`	`284`
`285`	`285`	`// all children and bodies have spent their content, so move to the footer`
`286`	`286`	`mNextSection = Footer;`
`287`		`- FALLTHROUGH;`
	`287`	`+ FALLTHROUGH`
`288`	`288`	`}`
`289`	`289`
`290`	`290`	`case Footer:`
`@@ -303,7 +303,7 @@ bool QgsAbstractReportSection::next()`
`303`	`303`	`}`
`304`	`304`
`305`	`305`	`// if not, then we're all done`
`306`		`- FALLTHROUGH;`
	`306`	`+ FALLTHROUGH`
`307`	`307`	`}`
`308`	`308`
`309`	`309`	`case End:`
Original file line number	Diff line number	Diff line change
`@@ -273,7 +273,7 @@ double QgsRenderContext::convertToPainterUnits( double size, QgsUnitTypes::Rende`
`273`	`273`	`size = convertMetersToMapUnits( size );`
`274`	`274`	`unit = QgsUnitTypes::RenderMapUnits;`
`275`	`275`	`// Fall through to RenderMapUnits with size in meters converted to size in MapUnits`
`276`		`- FALLTHROUGH;`
	`276`	`+ FALLTHROUGH`
`277`	`277`	`}`
`278`	`278`	`case QgsUnitTypes::RenderMapUnits:`
`279`	`279`	`{`
`@@ -323,7 +323,7 @@ double QgsRenderContext::convertToMapUnits( double size, QgsUnitTypes::RenderUni`
`323`	`323`	`{`
`324`	`324`	`size = convertMetersToMapUnits( size );`
`325`	`325`	`// Fall through to RenderMapUnits with values of meters converted to MapUnits`
`326`		`- FALLTHROUGH;`
	`326`	`+ FALLTHROUGH`
`327`	`327`	`}`
`328`	`328`	`case QgsUnitTypes::RenderMapUnits:`
`329`	`329`	`{`
Original file line number	Diff line number	Diff line change
`@@ -1813,7 +1813,7 @@ void QgsTextRenderer::drawPart( const QRectF &rect, double rotation, HAlignment`
`1813`	`1813`	`if ( !format.buffer().enabled() )`
`1814`	`1814`	`break;`
`1815`	`1815`	`}`
`1816`		`- FALLTHROUGH;`
	`1816`	`+ FALLTHROUGH`
`1817`	`1817`	`case Text:`
`1818`	`1818`	`case Shadow:`
`1819`	`1819`	`{`
`@@ -1857,7 +1857,7 @@ void QgsTextRenderer::drawPart( QPointF origin, double rotation, QgsTextRenderer`
`1857`	`1857`	`if ( !format.buffer().enabled() )`
`1858`	`1858`	`break;`
`1859`	`1859`	`}`
`1860`		`- FALLTHROUGH;`
	`1860`	`+ FALLTHROUGH`
`1861`	`1861`	`case Text:`
`1862`	`1862`	`case Shadow:`
`1863`	`1863`	`{`
`@@ -2077,6 +2077,7 @@ void QgsTextRenderer::drawBackground( QgsRenderContext &context, QgsTextRenderer`
`2077`	`2077`	`component.origin.y() - height / 2.0 + originAdjust );`
`2078`	`2078`	`break;`
`2079`	`2079`	`}`
	`2080`	`+ break;`
`2080`	`2081`	`}`
`2081`	`2082`
`2082`	`2083`	`case Label:`
Original file line number	Diff line number	Diff line change
`@@ -3464,7 +3464,7 @@ QSet<QVariant> QgsVectorLayer::uniqueValues( int index, int limit ) const`
`3464`	`3464`	`uniqueValues = mDataProvider->uniqueValues( index, limit );`
`3465`	`3465`	`return uniqueValues;`
`3466`	`3466`	`}`
`3467`		`- FALLTHROUGH;`
	`3467`	`+ FALLTHROUGH`
`3468`	`3468`	`//we need to go through each feature`
`3469`	`3469`	`case QgsFields::OriginJoin:`
`3470`	`3470`	`case QgsFields::OriginExpression:`
`@@ -3561,7 +3561,7 @@ QStringList QgsVectorLayer::uniqueStringsMatching( int index, const QString &sub`
`3561`	`3561`	`{`
`3562`	`3562`	`return mDataProvider->uniqueStringsMatching( index, substring, limit, feedback );`
`3563`	`3563`	`}`
`3564`		`- FALLTHROUGH;`
	`3564`	`+ FALLTHROUGH`
`3565`	`3565`	`//we need to go through each feature`
`3566`	`3566`	`case QgsFields::OriginJoin:`
`3567`	`3567`	`case QgsFields::OriginExpression:`
`@@ -3654,7 +3654,7 @@ QVariant QgsVectorLayer::minimumValue( int index ) const`
`3654`	`3654`	`return mDataProvider->minimumValue( index );`
`3655`	`3655`	`}`
`3656`	`3656`	`}`
`3657`		`- FALLTHROUGH;`
	`3657`	`+ FALLTHROUGH`
`3658`	`3658`	`// no choice but to go through all features`
`3659`	`3659`	`case QgsFields::OriginExpression:`
`3660`	`3660`	`case QgsFields::OriginJoin:`
`@@ -3740,7 +3740,7 @@ QVariant QgsVectorLayer::maximumValue( int index ) const`
`3740`	`3740`	`return mDataProvider->maximumValue( index );`
`3741`	`3741`	`}`
`3742`	`3742`
`3743`		`- FALLTHROUGH;`
	`3743`	`+ FALLTHROUGH`
`3744`	`3744`	`//no choice but to go through each feature`
`3745`	`3745`	`case QgsFields::OriginJoin:`
`3746`	`3746`	`case QgsFields::OriginExpression:`
Original file line number	Diff line number	Diff line change
`@@ -996,7 +996,7 @@ void QgsSymbol::renderFeature( const QgsFeature &feature, QgsRenderContext &cont`
`996`	`996`	`break;`
`997`	`997`	`}`
`998`	`998`
`999`		`- FALLTHROUGH;`
	`999`	`+ FALLTHROUGH`
`1000`	`1000`	`}`
`1001`	`1001`	`default:`
`1002`	`1002`	`QgsDebugMsg( QString( "feature %1: unsupported wkb type %2/%3 for rendering" )`
Original file line number	Diff line number	Diff line change
`@@ -395,7 +395,7 @@ void QgsGeorefPluginGui::generateGDALScript()`
`395`	`395`	`break;`
`396`	`396`	`}`
`397`	`397`	`}`
`398`		`- FALLTHROUGH;`
	`398`	`+ FALLTHROUGH`
`399`	`399`	`default:`
`400`	`400`	`mMessageBar->pushMessage( tr( "Invalid Transform" ), tr( "GDAL scripting is not supported for %1 transformation." )`
`401`	`401`	`.arg( convertTransformEnumToString( mTransformParam ) )`
Original file line number	Diff line number	Diff line change
`@@ -322,7 +322,7 @@ int QgsGrassNewMapset::nextId() const`
`322`	`322`	`id = MapSet;`
`323`	`323`	`break;`
`324`	`324`	`}`
`325`		`- FALLTHROUGH;`
	`325`	`+ FALLTHROUGH`
`326`	`326`	`case Database:`
`327`	`327`	`case Crs:`
`328`	`328`	`case Region:`
Original file line number	Diff line number	Diff line change
`@@ -1619,6 +1619,8 @@ bool QgsOgrProvider::addAttributeOGRLevel( const QgsField &field, bool &ignoreEr`
`1619`	`1619`	`{`
`1620`	`1620`	`case QVariant::Bool:`
`1621`	`1621`	`OGR_Fld_SetSubType( fielddefn.get(), OFSTBoolean );`
	`1622`	`+ break;`
	`1623`	`+`
`1622`	`1624`	`default:`
`1623`	`1625`	`break;`
`1624`	`1626`	`}`