aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorYordan Karadzhov (VMware) <y.karadz@gmail.com>2019-08-19 12:27:22 +0300
committerYordan Karadzhov (VMware) <y.karadz@gmail.com>2019-08-19 12:58:52 +0300
commit10f37d7d916fb9c3426b95bdc6b32378fb78b067 (patch)
treef20d5537f646edbfd9f0b7138345bbe2b6afdf40
parentb7521a8fd247188d62943d5ddd98a93ae41faffc (diff)
downloadtrace-cmd-10f37d7d916fb9c3426b95bdc6b32378fb78b067.tar.gz
kernel-shark: Optimize the error messages when "Record" fails to start
Removing duplicate code. Signed-off-by: Yordan Karadzhov (VMware) <y.karadz@gmail.com>
-rw-r--r--kernel-shark/src/KsMainWindow.cpp14
-rw-r--r--kernel-shark/src/KsMainWindow.hpp2
2 files changed, 9 insertions, 7 deletions
diff --git a/kernel-shark/src/KsMainWindow.cpp b/kernel-shark/src/KsMainWindow.cpp
index 73d68777..651cf592 100644
--- a/kernel-shark/src/KsMainWindow.cpp
+++ b/kernel-shark/src/KsMainWindow.cpp
@@ -1199,18 +1199,18 @@ void KsMainWindow::_captureFinished(int ret, QProcess::ExitStatus st)
return;
}
- if (ret != 0 && st == QProcess::NormalExit) {
- QString message = "Capture process failed:<br>";
-
- message += capture->errorString();
- message += capture->readAllStandardError();
- _error(message, "captureFinishedErr", false, false);
- }
+ if (ret != 0 && st == QProcess::NormalExit)
+ _captureErrorMessage(capture);
}
void KsMainWindow::_captureError(QProcess::ProcessError error)
{
QProcess *capture = (QProcess *)sender();
+ _captureErrorMessage(capture);
+}
+
+void KsMainWindow::_captureErrorMessage(QProcess *capture)
+{
QString message = "Capture process failed:<br>";
message += capture->errorString();
diff --git a/kernel-shark/src/KsMainWindow.hpp b/kernel-shark/src/KsMainWindow.hpp
index 4a7b8ab6..eef4f96c 100644
--- a/kernel-shark/src/KsMainWindow.hpp
+++ b/kernel-shark/src/KsMainWindow.hpp
@@ -215,6 +215,8 @@ private:
void _captureError(QProcess::ProcessError error);
+ void _captureErrorMessage(QProcess *capture);
+
void _readSocket();
void _splitterMoved(int pos, int index);