Whitespace and dead code cleanup

The coding style for switch isn't the prettiest, but let's at least keep
things consistent.

Also, those ugly break statements never got reached, anyway.

And there was no need to re-declare and re-generate currentString.

Signed-off-by: Dirk Hohndel <dirk@hohndel.org>

1 files changed, 6 insertions, 8 deletions

diff --git a/qt-ui/models.cpp b/qt-ui/models.cpp
index 93bc446ef..942be2e5a 100644
--- a/qt-ui/models.cpp
+++ b/qt-ui/models.cpp
@@ -1862,14 +1862,12 @@ QVariant LanguageModel::data(const QModelIndex& index, int role) const
 	if (!index.isValid())
 		return QVariant();
 	switch (role) {
-		case Qt::DisplayRole: {
-			QLocale l( currentString.remove("subsurface_"));
-			return currentString == "English" ? currentString : QString("%1 (%2)").arg(l.languageToString(l.language())).arg(l.countryToString(l.country()));
-		}break;
-	case Qt::UserRole: {
-			QString currentString = languages.at(index.row());
-			return currentString == "English" ? "en_US" : currentString.remove("subsurface_");
-		}break;
+	case Qt::DisplayRole: {
+		QLocale l( currentString.remove("subsurface_"));
+		return currentString == "English" ? currentString : QString("%1 (%2)").arg(l.languageToString(l.language())).arg(l.countryToString(l.country()));
+	}
+	case Qt::UserRole:
+		return currentString == "English" ? "en_US" : currentString.remove("subsurface_");
 	}
 	return QVariant();
 }