From 5a0b9ba19af14de65c605b76d739ba8b1e13edea Mon Sep 17 00:00:00 2001
From: bluebrother <bluebrother@a1c6a512-1295-4272-9138-f99709370657>
Date: Fri, 7 May 2010 16:18:41 +0000
Subject: [PATCH] Add repeat counting to system trace.

This make the system trace check for repeating lines, and only print a "(last
message repeated n lines.)" instead of the real message multiple times. This
keeps the trace much shorter if messages are repeated. The drawback is that the
replacement count message will only get printed on the next line getting
traced, so until that happens it swallows the repeated lines.
Before saving the systrace buffer is flushed, so this should not raise a problem.

git-svn-id: svn://svn.rockbox.org/rockbox/trunk@25878 a1c6a512-1295-4272-9138-f99709370657
---
 rbutil/rbutilqt/systrace.cpp | 29 ++++++++++++++++++++++++++---
 rbutil/rbutilqt/systrace.h   |  3 +++
 2 files changed, 29 insertions(+), 3 deletions(-)

diff --git a/rbutil/rbutilqt/systrace.cpp b/rbutil/rbutilqt/systrace.cpp
index 1b9de6b71..a1e8bd023 100644
--- a/rbutil/rbutilqt/systrace.cpp
+++ b/rbutil/rbutilqt/systrace.cpp
@@ -24,6 +24,8 @@
 #include "rbsettings.h"
 
 QString SysTrace::debugbuffer;
+QString SysTrace::lastmessage;
+unsigned int SysTrace::repeat = 0;
 
 SysTrace::SysTrace(QWidget *parent) : QDialog(parent)
 {
@@ -41,6 +43,7 @@ SysTrace::SysTrace(QWidget *parent) : QDialog(parent)
 void SysTrace::refresh(void)
 {
     int pos = ui.textTrace->verticalScrollBar()->value();
+    flush();
     ui.textTrace->setHtml("<pre>" + debugbuffer + "</pre>");
     ui.textTrace->verticalScrollBar()->setValue(pos);
     QString oldlog = RbSettings::value(RbSettings::CachePath).toString()
@@ -54,6 +57,8 @@ void SysTrace::save(QString filename)
     if(filename.isEmpty())
         filename = RbSettings::value(RbSettings::CachePath).toString()
                     + "/rbutil-trace.log";
+    // make sure any repeat detection is flushed
+    flush();
     // append save date to the trace. Append it directly instead of using
     // qDebug() as the handler might have been unregistered.
     debugbuffer.append("[SysTrace] saving trace at ");
@@ -91,11 +96,29 @@ void SysTrace::savePreviousTrace(void)
 void SysTrace::debug(QtMsgType type, const char* msg)
 {
     (void)type;
-    debugbuffer.append(msg);
-    debugbuffer.append("\n");
+    if(lastmessage != msg) {
+        lastmessage = msg;
+        flush();
+        debugbuffer.append(msg);
+        debugbuffer.append("\n");
 #if !defined(NODEBUG)
-    fprintf(stderr, "%s\n", msg);
+        fprintf(stderr, "%s\n", msg);
 #endif
+        repeat = 1;
+    }
+    else {
+        repeat++;
+    }
+}
 
+void SysTrace::flush(void)
+{
+    if(repeat > 1) {
+        debugbuffer.append(
+                QString("    (Last message repeasted %1 times.)\n").arg(repeat));
+#if !defined(NODEBUG)
+        fprintf(stderr, "    (Last message repeated %i times.)\n", repeat);
+#endif
+    }
 }
 
diff --git a/rbutil/rbutilqt/systrace.h b/rbutil/rbutilqt/systrace.h
index 4e6206a88..f15e33c14 100644
--- a/rbutil/rbutilqt/systrace.h
+++ b/rbutil/rbutilqt/systrace.h
@@ -34,8 +34,11 @@ class SysTrace : public QDialog
         static QString getTrace() {return debugbuffer;}
         static void save(QString filename = "");
     private:
+        static void flush(void);
         Ui::SysTraceFrm ui;
         static QString debugbuffer;
+        static QString lastmessage;
+        static unsigned int repeat;
 
     private slots:
         void saveCurrentTrace(void);
-- 
2.11.4.GIT