Another braces warning this that I overlooked

Guido Lerch guido.lerch at gmail.com
Wed Oct 21 15:45:47 PDT 2015


-- 
Best regards,
Guido
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.subsurface-divelog.org/pipermail/subsurface/attachments/20151022/47dadbdd/attachment.html>
-------------- next part --------------
From 6121bdb8543fd067188e793837e577edef912ede Mon Sep 17 00:00:00 2001
From: Guido Lerch <guido.lerch at gmail.com>
Date: Thu, 22 Oct 2015 00:42:30 +0200
Subject: [PATCH 2/2] Fixing warnings about braces in cylindermodel.cpp-2

Morace braces needed.Sorry

Signed-off-by: Guido Lerch <guido.lerch at gmail.com>
---
 qt-models/cylindermodel.cpp | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/qt-models/cylindermodel.cpp b/qt-models/cylindermodel.cpp
index c697145..7992545 100644
--- a/qt-models/cylindermodel.cpp
+++ b/qt-models/cylindermodel.cpp
@@ -111,20 +111,22 @@ QVariant CylindersModel::data(const QModelIndex &index, int role) const
 		}
 		break;
 	case Qt::DecorationRole:
-		if (index.column() == REMOVE)
+		if (index.column() == REMOVE) {
 			if (rowCount() > 1) {
 				ret = trashIcon();
 			} else {
 				ret = trashForbiddenIcon();
 			}
+		}
 		break;
 	case Qt::SizeHintRole:
-		if (index.column() == REMOVE)
+		if (index.column() == REMOVE) {
 			if (rowCount() > 1) {
 				ret = trashIcon();
 			} else {
 				ret = trashForbiddenIcon();
 			}
+		}
 		break;
 
 	case Qt::ToolTipRole:
-- 
2.3.8 (Apple Git-58)



More information about the subsurface mailing list