Simplify switch statement with an if statement

f8e06d55 · Kurt Hindenburg · 9a1c47e7 · f8e06d55
Commit f8e06d55 authored Jul 22, 2020 by Kurt Hindenburg
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 7 deletions

src/filterHotSpots/UrlFilterHotspot.cpp src/filterHotSpots/UrlFilterHotspot.cpp +3 -7

No files found.
--- a/src/filterHotSpots/UrlFilterHotspot.cpp
+++ b/src/filterHotSpots/UrlFilterHotspot.cpp
@@ -43,14 +43,10 @@ UrlFilterHotSpot::UrlFilterHotSpot(int startLine, int startColumn, int endLine,
                            const QStringList &capturedTexts) :
    RegExpFilterHotSpot(startLine, startColumn, endLine, endColumn, capturedTexts)
 {
-    switch(urlType()) {
-    case Email:
+    const UrlType kind = urlType();
+    if (kind == Email) {
        setType(EMailAddress);
-        break;
-
-    case StandardUrl:
-    case Unknown:
-    default:
+    } else {
        setType(Link);
    }
 }