diff options
| author |  Dirk Hohndel <dirk@hohndel.org> | 2014-02-12 06:18:15 -0800 |
|---|---|---|
| committer | Dirk Hohndel <dirk@hohndel.org> | 2014-02-12 06:22:10 -0800 |
| commit | 6332976c3d570e7f585e17b6897ae3dc07d7376c (patch) | |
| tree | 818266e840fad0a9b92f74fc9a71833ac2c8758e /qt-ui | |
| parent | 9c9867dec527e3d0231a71c043fd9dc9b894a339 (diff) | |
| download | subsurface-6332976c3d570e7f585e17b6897ae3dc07d7376c.tar.gz | |
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>
Diffstat (limited to 'qt-ui')
| -rw-r--r-- | qt-ui/graphicsview-common.cpp | 2 | ||||
| -rw-r--r-- | qt-ui/models.cpp | 14 |
2 files changed, 7 insertions, 9 deletions
diff --git a/qt-ui/graphicsview-common.cpp b/qt-ui/graphicsview-common.cpp index 4d96aa8c6..e23691f3f 100644 --- a/qt-ui/graphicsview-common.cpp +++ b/qt-ui/graphicsview-common.cpp @@ -74,4 +74,4 @@ QColor getSacColor(int sac, int avg_sac) if (sac_index > SAC_COLORS - 1) sac_index = SAC_COLORS - 1; return getColor((color_indice_t)(SAC_COLORS_START_IDX + sac_index), false); -}
\ No newline at end of file +} 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(); } |