diff options
| author |  Dirk Hohndel <dirk@hohndel.org> | 2019-10-27 07:26:50 -0400 |
|---|---|---|
| committer | Dirk Hohndel <dirk@hohndel.org> | 2019-10-27 07:35:46 -0700 |
| commit | 08fcda60ac9a5705fa84b79cabe189a9e850e31d (patch) | |
| tree | 1371e7ca59a3d37140f778a0a797aad7b9522ac2 /qt-models/weightmodel.cpp | |
| parent | 80f76773c1940b94a907a76c8d42b60470b33d3d (diff) | |
| download | subsurface-08fcda60ac9a5705fa84b79cabe189a9e850e31d.tar.gz | |
Cleanup: add missing break statements
The current code of course works just fine. But there's a risk someone might
add something to one of these cases and not realize that there is an implicit
fall through going on. This is cleaner.
Found by Coverity. Fixes CID 350079
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
Diffstat (limited to 'qt-models/weightmodel.cpp')
| -rw-r--r-- | qt-models/weightmodel.cpp | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/qt-models/weightmodel.cpp b/qt-models/weightmodel.cpp index 715af1571..3b5a122bf 100644 --- a/qt-models/weightmodel.cpp +++ b/qt-models/weightmodel.cpp @@ -63,12 +63,15 @@ QVariant WeightModel::data(const QModelIndex &index, int role) const case Qt::DecorationRole: if (index.column() == REMOVE) return trashIcon(); + break; case Qt::SizeHintRole: if (index.column() == REMOVE) return trashIcon().size(); + break; case Qt::ToolTipRole: if (index.column() == REMOVE) return tr("Clicking here will remove this weight system."); + break; } return QVariant(); } |