Process stderr one line at time.

Instead of reading all into memory.

Benefit is probably only marginal, but code is equally lean.

diff --git a/gui/retracer.cpp b/gui/retracer.cpp
index 865f61fcb5d1004822428e2cff915e31de76d542..fa220117f0c5bb9de24d490d0b5d5c07f2ed7e53 100644 (file)
--- a/gui/retracer.cpp
+++ b/gui/retracer.cpp
@@ -255,11 +255,11 @@ void RetraceProcess::replayFinished(int exitCode, QProcess::ExitStatus exitStatu
         }
     }
 
-    QString errStr = m_process->readAllStandardError();
-    QStringList errorLines = errStr.split('\n');
+    m_process->setReadChannel(QProcess::StandardError);
     QList<ApiTraceError> errors;
-    QRegExp regexp("(^\\d+): +(\\b\\w+\\b): (.+$)");
-    foreach(QString line, errorLines) {
+    QRegExp regexp("(^\\d+): +(\\b\\w+\\b): ([^\\r\\n]+)[\\r\\n]*$");
+    while (!m_process->atEnd()) {
+        QString line = m_process->readLine();
         if (regexp.indexIn(line) != -1) {
             ApiTraceError error;
             error.callIndex = regexp.cap(1).toInt();