From aac8c8f8de020d51627fab6f52004468c82863f9 Mon Sep 17 00:00:00 2001
From: Lieven Hey <lieven.hey@kdab.com>
Date: Tue, 17 Sep 2024 13:22:51 +0200
Subject: [PATCH] feat: open more perf files

Currently `hotspot dir` only opens perf.data files. This patch extends
this to perf.data, perf.data.old and perf.data.perfparser files.
---
 src/main.cpp | 30 ++++++++++++++++++++++++++----
 1 file changed, 26 insertions(+), 4 deletions(-)

diff --git a/src/main.cpp b/src/main.cpp
index 58e46ca8d..c368dad3e 100644
--- a/src/main.cpp
+++ b/src/main.cpp
@@ -57,6 +57,20 @@ void Q_DECL_UNUSED initRCCIconTheme()
 }
 #endif
 
+namespace {
+QStringList findPerfDataFiles(const QDir& directory = QDir::current())
+{
+    QStringList files;
+
+    for (const auto& filename : {QStringLiteral("perf.data"), QStringLiteral("perf.data.perfparser")}) {
+        if (directory.exists(filename)) {
+            files.push_back(directory.absoluteFilePath(filename));
+        }
+    }
+    return files;
+}
+}
+
 std::unique_ptr<QCoreApplication> createApplication(int& argc, char* argv[])
 {
     const std::initializer_list<std::string_view> nonGUIOptions = {"--version", "-v", "--exportTo",
@@ -236,7 +250,12 @@ int main(int argc, char** argv)
     if (!files.isEmpty()) {
         auto file = files.constFirst();
         if (QFileInfo(file).isDir()) {
-            file.append(QLatin1String("/perf.data"));
+            // search for common perf data files in dir and open the first one found
+            // if none is found, hotspot will complain, that `file` is not a file
+            const auto perfDataFiles = findPerfDataFiles(QDir(file));
+            if (!perfDataFiles.isEmpty()) {
+                file = perfDataFiles.first();
+            }
         }
 
         if (parser.isSet(exportTo)) {
@@ -267,9 +286,12 @@ int main(int argc, char** argv)
     } else {
         // open perf.data in current CWD, if it exists
         // this brings hotspot closer to the behavior of "perf report"
-        const auto perfDataFile = QStringLiteral("perf.data");
-        if (QFile::exists(perfDataFile) && window) {
-            window->openFile(perfDataFile);
+        const auto perfDataFiles = findPerfDataFiles();
+        for (const auto& perfDataFile : perfDataFiles) {
+            if (window) {
+                window->openFile(perfDataFile);
+                break;
+            }
         }
     }
     if (window)