reordered checks in brushSystem

GitMensch · Nov 1, 2023 · 89c3808 · 89c3808
1 parent ea54cbf
commit 89c3808
Showing 1 changed file with 2 additions and 5 deletions.
diff --git a/src/flamegraph.cpp b/src/flamegraph.cpp
@@ -379,13 +379,10 @@ QBrush brushSystem(const Data::Symbol& symbol)
     static const auto user = QBrush(QColor(200, 200, 0, 125));
     static const auto unknown = QBrush(QColor(50, 50, 50, 125));
 
-    // I have seen [ only on kernel calls
+    // remark lievenhey: I have seen [ only on kernel calls
     if (symbol.path.isEmpty() || symbol.path.startsWith(QLatin1Char('['))) {
         return unknown;
-    } else if (isSystemPath(symbol.path)) {
-        if (isUserPath(symbol.path)) {
-            return user;
-        }
+    } else if (!isUserPath(symbol.path) && isSystemPath(symbol.path)) {
         return system;
     }
     return user;