From 08fcda60ac9a5705fa84b79cabe189a9e850e31d Mon Sep 17 00:00:00 2001 From: Dirk Hohndel Date: Sun, 27 Oct 2019 07:26:50 -0400 Subject: 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 --- qt-models/weightmodel.cpp | 3 +++ 1 file changed, 3 insertions(+) (limited to 'qt-models/weightmodel.cpp') 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(); } -- cgit v1.2.3-70-g09d2