From 2950477a01cb89c84da3a75430a8b58b09bd3437 Mon Sep 17 00:00:00 2001
From: Vladimir Byko-Ianko <v.bykoianko@corp.mail.ru>
Date: Wed, 13 Feb 2019 14:39:21 +0300
Subject: [PATCH] Review fixes.

---
 .../openlr_assessment_tool/traffic_mode.cpp   | 25 ++++++++-----------
 1 file changed, 10 insertions(+), 15 deletions(-)

diff --git a/openlr/openlr_match_quality/openlr_assessment_tool/traffic_mode.cpp b/openlr/openlr_match_quality/openlr_assessment_tool/traffic_mode.cpp
index bfed2d2351..0007e5718c 100644
--- a/openlr/openlr_match_quality/openlr_assessment_tool/traffic_mode.cpp
+++ b/openlr/openlr_match_quality/openlr_assessment_tool/traffic_mode.cpp
@@ -265,22 +265,17 @@ QVariant TrafficMode::data(const QModelIndex & index, int role) const
 QVariant TrafficMode::headerData(int section, Qt::Orientation orientation,
                                  int role /* = Qt::DisplayRole */) const
 {
-  if (orientation == Qt::Horizontal)
-  {
-    if (role == Qt::DisplayRole)
-    {
-      switch (section)
-      {
-      case 0: return "Segment id"; break;
-      case 1: return "Status code"; break;
-      case 2: return "Positive offset (Meters)"; break;
-      case 3: return "Negative offset (Meters)"; break;
-      }
-      UNREACHABLE();
-    }
-  }
+  if (orientation != Qt::Horizontal && role != Qt::DisplayRole)
+    return QVariant();
 
-  return QVariant();
+  switch (section)
+  {
+  case 0: return "Segment id"; break;
+  case 1: return "Status code"; break;
+  case 2: return "Positive offset (Meters)"; break;
+  case 3: return "Negative offset (Meters)"; break;
+  }
+  UNREACHABLE();
 }
 
 void TrafficMode::OnItemSelected(QItemSelection const & selected, QItemSelection const &)