QmlProfilerTraceClient: Cleanup

Move private members into d pointer, remove unused signal,
rename clearView() method & clear() signal to clearData(), dataCleared()

Change-Id: I0baf78107861a469185c736fc448b4d3a7929cdd
Reviewed-on: http://codereview.qt.nokia.com/2691
Reviewed-by: Kai Koehne <kai.koehne@nokia.com>

src/libs/qmljsdebugclient/qmlprofilertraceclient.cpp

@@ -32,29 +32,64 @@
 
 #include "qmlprofilertraceclient.h"
 
+namespace QmlJsDebugClient {
+
+class QmlProfilerTraceClientPrivate {
+public:
+    QmlProfilerTraceClientPrivate()
+        : inProgressRanges(0)
+        , maximumTime(0)
+        , recording(false)
+        , nestingLevel(0)
+    {
+        ::memset(rangeCount, 0, MaximumQmlEventType * sizeof(int));
+        ::memset(nestingInType, 0, MaximumQmlEventType * sizeof(int));
+    }
+
+    qint64 inProgressRanges;
+    QStack<qint64> rangeStartTimes[MaximumQmlEventType];
+    QStack<QStringList> rangeDatas[MaximumQmlEventType];
+    QStack<Location> rangeLocations[MaximumQmlEventType];
+    int rangeCount[MaximumQmlEventType];
+    qint64 maximumTime;
+    bool recording;
+    int nestingLevel;
+    int nestingInType[MaximumQmlEventType];
+};
+
+} // namespace QmlJsDebugClient
+
 using namespace QmlJsDebugClient;
 
 static const int GAP_TIME = 150;
 
 QmlProfilerTraceClient::QmlProfilerTraceClient(QDeclarativeDebugConnection *client)
-    : QDeclarativeDebugClient(QLatin1String("CanvasFrameRate"), client),
+    : QDeclarativeDebugClient(QLatin1String("CanvasFrameRate"), client)
-      m_inProgressRanges(0), m_maximumTime(0), m_recording(false), m_nestingLevel(0)
+    , d(new QmlProfilerTraceClientPrivate)
 {
-    ::memset(m_rangeCount, 0, MaximumQmlEventType * sizeof(int));
-    ::memset(m_nestingInType, 0, MaximumQmlEventType * sizeof(int));
 }
 
-void QmlProfilerTraceClient::clearView()
+QmlProfilerTraceClient::~QmlProfilerTraceClient()
 {
-    ::memset(m_rangeCount, 0, MaximumQmlEventType * sizeof(int));
+    delete d;
-    ::memset(m_nestingInType, 0, MaximumQmlEventType * sizeof(int));
+}
-    m_nestingLevel = 0;
+
-    emit clear();
+void QmlProfilerTraceClient::clearData()
+{
+    ::memset(d->rangeCount, 0, MaximumQmlEventType * sizeof(int));
+    ::memset(d->nestingInType, 0, MaximumQmlEventType * sizeof(int));
+    d->nestingLevel = 0;
+    emit cleared();
+}
+
+bool QmlProfilerTraceClient::isRecording() const
+{
+    return d->recording;
 }
 
 void QmlProfilerTraceClient::setRecording(bool v)
 {
-    if (v == m_recording)
+    if (v == d->recording)
         return;
 
     if (status() == Enabled) {
@@ -64,15 +99,15 @@ void QmlProfilerTraceClient::setRecording(bool v)
         sendMessage(ba);
     }
 
-    m_recording = v;
+    d->recording = v;
     emit recordingChanged(v);
 }
 
 void QmlProfilerTraceClient::statusChanged(Status status)
 {
     if (status == Enabled) {
-        m_recording = !m_recording;
+        d->recording = !d->recording;
-        setRecording(!m_recording);
+        setRecording(!d->recording);
         emit enabled();
     }
 }
@@ -92,7 +127,7 @@ void QmlProfilerTraceClient::messageReceived(const QByteArray &data)
     if (messageType >= MaximumMessage)
         return;
 
-    if (time > (m_maximumTime + GAP_TIME) && 0 == m_inProgressRanges)
+    if (time > (d->maximumTime + GAP_TIME) && 0 == d->inProgressRanges)
         emit gap(time);
 
     if (messageType == Event) {
@@ -101,7 +136,7 @@ void QmlProfilerTraceClient::messageReceived(const QByteArray &data)
 
         if (event < MaximumEventType) {
             emit this->event((EventType)event, time);
-            m_maximumTime = qMax(time, m_maximumTime);
+            d->maximumTime = qMax(time, d->maximumTime);
         }
     } else if (messageType == Complete) {
         emit complete();
@@ -113,20 +148,20 @@ void QmlProfilerTraceClient::messageReceived(const QByteArray &data)
             return;
 
         if (messageType == RangeStart) {
-            m_rangeStartTimes[range].push(time);
+            d->rangeStartTimes[range].push(time);
-            m_inProgressRanges |= (static_cast<qint64>(1) << range);
+            d->inProgressRanges |= (static_cast<qint64>(1) << range);
-            ++m_rangeCount[range];
+            ++d->rangeCount[range];
-            ++m_nestingLevel;
+            ++d->nestingLevel;
-            ++m_nestingInType[range];
+            ++d->nestingInType[range];
         } else if (messageType == RangeData) {
             QString data;
             stream >> data;
 
-            int count = m_rangeCount[range];
+            int count = d->rangeCount[range];
             if (count > 0) {
-                while (m_rangeDatas[range].count() < count)
+                while (d->rangeDatas[range].count() < count)
-                    m_rangeDatas[range].push(QStringList());
+                    d->rangeDatas[range].push(QStringList());
-                m_rangeDatas[range][count-1] << data;
+                d->rangeDatas[range][count-1] << data;
             }
 
         } else if (messageType == RangeLocation) {
@@ -134,28 +169,28 @@ void QmlProfilerTraceClient::messageReceived(const QByteArray &data)
             int line;
             stream >> fileName >> line;
 
-            if (m_rangeCount[range] > 0) {
+            if (d->rangeCount[range] > 0) {
-                m_rangeLocations[range].push(Location(fileName, line));
+                d->rangeLocations[range].push(Location(fileName, line));
             }
         } else {
-            if (m_rangeCount[range] > 0) {
+            if (d->rangeCount[range] > 0) {
-                --m_rangeCount[range];
+                --d->rangeCount[range];
-                if (m_inProgressRanges & (static_cast<qint64>(1) << range))
+                if (d->inProgressRanges & (static_cast<qint64>(1) << range))
-                    m_inProgressRanges &= ~(static_cast<qint64>(1) << range);
+                    d->inProgressRanges &= ~(static_cast<qint64>(1) << range);
 
-                m_maximumTime = qMax(time, m_maximumTime);
+                d->maximumTime = qMax(time, d->maximumTime);
-                QStringList data = m_rangeDatas[range].count() ? m_rangeDatas[range].pop() : QStringList();
+                QStringList data = d->rangeDatas[range].count() ? d->rangeDatas[range].pop() : QStringList();
-                Location location = m_rangeLocations[range].count() ? m_rangeLocations[range].pop() : Location();
+                Location location = d->rangeLocations[range].count() ? d->rangeLocations[range].pop() : Location();
 
-                qint64 startTime = m_rangeStartTimes[range].pop();
+                qint64 startTime = d->rangeStartTimes[range].pop();
-                emit this->range((QmlEventType)range, m_nestingLevel, m_nestingInType[range], startTime,
+                emit this->range((QmlEventType)range, d->nestingLevel, d->nestingInType[range], startTime,
                                  time - startTime, data, location.fileName, location.line);
-                --m_nestingLevel;
+                --d->nestingLevel;
-                --m_nestingInType[range];
+                --d->nestingInType[range];
-                if (m_rangeCount[range] == 0) {
+                if (d->rangeCount[range] == 0) {
-                    int count = m_rangeDatas[range].count() +
+                    int count = d->rangeDatas[range].count() +
-                                m_rangeStartTimes[range].count() +
+                                d->rangeStartTimes[range].count() +
-                                m_rangeLocations[range].count();
+                                d->rangeLocations[range].count();
                     if (count != 0)
                         qWarning() << "incorrectly nested data";
                 }

src/libs/qmljsdebugclient/qmlprofilertraceclient.h

@@ -59,7 +59,8 @@ class QMLJSDEBUGCLIENT_EXPORT QmlProfilerTraceClient : public QmlJsDebugClient::
     using QObject::event;
 
 public:
-    QmlProfilerTraceClient(QmlJsDebugClient::QDeclarativeDebugConnection *client);
+    QmlProfilerTraceClient(QDeclarativeDebugConnection *client);
+    ~QmlProfilerTraceClient();
 
     enum EventType {
         FramePaint,
@@ -80,40 +81,30 @@ public:
         MaximumMessage
     };
 
-    bool isRecording() const { return m_recording; }
+    bool isRecording() const;
 
 public slots:
     void setRecording(bool);
-    void clearView();
+    void clearData();
 
 signals:
     void complete();
-    void gap(qint64);
+    void gap(qint64 time);
     void event(int event, qint64 time);
     void range(int type, int nestingLevel, int nestingInType, qint64 startTime, qint64 length,
                const QStringList &data, const QString &fileName, int line);
 
-    void sample(int, int, int, bool);
-
     void recordingChanged(bool arg);
 
     void enabled();
-    void clear();
+    void cleared();
 
 protected:
     virtual void statusChanged(Status);
     virtual void messageReceived(const QByteArray &);
 
 private:
-    qint64 m_inProgressRanges;
+    class QmlProfilerTraceClientPrivate *d;
-    QStack<qint64> m_rangeStartTimes[MaximumQmlEventType];
-    QStack<QStringList> m_rangeDatas[MaximumQmlEventType];
-    QStack<Location> m_rangeLocations[MaximumQmlEventType];
-    int m_rangeCount[MaximumQmlEventType];
-    qint64 m_maximumTime;
-    bool m_recording;
-    int m_nestingLevel;
-    int m_nestingInType[MaximumQmlEventType];
 };
 
 } // namespace QmlJsDebugClient

src/plugins/qmlprofiler/qml/MainView.qml

@@ -163,7 +163,7 @@ Rectangle {
             }
         }
 
-        onClear: {
+        onDataCleared: {
             root.clearAll();
         }
 

src/plugins/qmlprofiler/tracewindow.cpp

@@ -154,7 +154,7 @@ void TraceWindow::updateTimer()
 void TraceWindow::clearDisplay()
 {
     if (m_plugin)
-        m_plugin.data()->clearView();
+        m_plugin.data()->clearData();
     else
         emit internalClearDisplay();
 }