diff options
author | Thiago Macieira <thiago.macieira@intel.com> | 2017-06-02 21:09:41 -0700 |
---|---|---|
committer | Thiago Macieira <thiago.macieira@intel.com> | 2017-06-06 06:28:47 +0000 |
commit | 1a7ed8d683334f473748e84a7413d38a03ca8434 (patch) | |
tree | b21c5b6c50e2d3cbdbc4258210b1f23bf85df3d7 | |
parent | 4417378dbb3a5b83c5c7a9563880b83ca0b52542 (diff) | |
download | qtlocation-1a7ed8d683334f473748e84a7413d38a03ca8434.tar.gz |
Add unnecessary break to keep GCC 7 happy
Falling through or breaking have the same effect, since the next case
label repeats the same condition in the "if", which must necessarily be
false for the code to have fallen through.
qdeclarativesearchresultmodel.cpp:632:9: error: this statement may fall through [-Werror=implicit-fallthrough=]
Change-Id: Ia3e896da908f42939148fffd14c48127381f219d
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
-rw-r--r-- | src/location/declarativeplaces/qdeclarativesearchresultmodel.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/location/declarativeplaces/qdeclarativesearchresultmodel.cpp b/src/location/declarativeplaces/qdeclarativesearchresultmodel.cpp index 31d152db..3450e3ea 100644 --- a/src/location/declarativeplaces/qdeclarativesearchresultmodel.cpp +++ b/src/location/declarativeplaces/qdeclarativesearchresultmodel.cpp @@ -631,6 +631,7 @@ QVariant QDeclarativeSearchResultModel::data(const QModelIndex &index, int role) case PlaceRole: if (result.type() == QPlaceSearchResult::PlaceResult) return QVariant::fromValue(static_cast<QObject *>(m_places.at(index.row()))); + break; case SponsoredRole: if (result.type() == QPlaceSearchResult::PlaceResult) { QPlaceResult placeResult = result; |