use switch/case in QCanBusFrame::toString() instead of multiple if's

Change-Id: I5c2940a37aa6d4261eb34ca8c90199905ba45868
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>

1 files changed, 7 insertions, 5 deletions

diff --git a/src/serialbus/qcanbusframe.cpp b/src/serialbus/qcanbusframe.cpp
index 1787c4c..787975a 100644
--- a/src/serialbus/qcanbusframe.cpp
+++ b/src/serialbus/qcanbusframe.cpp
@@ -321,14 +321,16 @@ QString QCanBusFrame::toString() const
 {
     const FrameType type = frameType();
 
-    if (type == InvalidFrame)
+    switch (type) {
+    case InvalidFrame:
         return QStringLiteral("(Invalid)");
-
-    if (type == ErrorFrame)
+    case ErrorFrame:
         return QStringLiteral("(Error)");
-
-    if (type == UnknownFrame)
+    case UnknownFrame:
         return QStringLiteral("(Unknown)");
+    default:
+        break;
+    }
 
     const char *idFormat = hasExtendedFrameFormat() ? "%08X" : "     %03X";
     const char *dlcFormat = hasFlexibleDataRateFormat() ? "  [%02d]" : "   [%d]";