diff options
author | Kai Koehne <kai.koehne@nokia.com> | 2011-11-11 14:48:12 +0100 |
---|---|---|
committer | Qt by Nokia <qt-info@nokia.com> | 2011-11-11 16:39:03 +0100 |
commit | 2557ff5a940242b398dee65c3c79cec088164e32 (patch) | |
tree | dc8d5bd105d66d97ed914f308444401cad530d3c /src/declarative/debugger | |
parent | 40360f040d9aa110997b375a460de5e6e4203ad6 (diff) |
Debugger: Remove option to constantly send tracing data
Change-Id: I392277453aa34498549e39182accf7797a262504
Reviewed-by: Christiaan Janssen <christiaan.janssen@nokia.com>
Diffstat (limited to 'src/declarative/debugger')
-rw-r--r-- | src/declarative/debugger/qdeclarativedebugtrace.cpp | 34 | ||||
-rw-r--r-- | src/declarative/debugger/qdeclarativedebugtrace_p.h | 1 |
2 files changed, 14 insertions, 21 deletions
diff --git a/src/declarative/debugger/qdeclarativedebugtrace.cpp b/src/declarative/debugger/qdeclarativedebugtrace.cpp index c7987d704f..b497f6a6ee 100644 --- a/src/declarative/debugger/qdeclarativedebugtrace.cpp +++ b/src/declarative/debugger/qdeclarativedebugtrace.cpp @@ -55,7 +55,7 @@ QT_BEGIN_NAMESPACE Q_GLOBAL_STATIC(QDeclarativeDebugTrace, traceInstance); // convert to a QByteArray that can be sent to the debug client -// use of QDataStream can skew results if m_deferredSend == false +// use of QDataStream can skew results // (see tst_qdeclarativedebugtrace::trace() benchmark) QByteArray QDeclarativeDebugData::toByteArray() const { @@ -75,7 +75,7 @@ QByteArray QDeclarativeDebugData::toByteArray() const QDeclarativeDebugTrace::QDeclarativeDebugTrace() : QDeclarativeDebugService(QLatin1String("CanvasFrameRate")), - m_enabled(false), m_deferredSend(true), m_messageReceived(false) + m_enabled(false), m_messageReceived(false) { m_timer.start(); if (status() == Enabled) { @@ -235,11 +235,7 @@ void QDeclarativeDebugTrace::animationFrameImpl(qint64 delta) void QDeclarativeDebugTrace::processMessage(const QDeclarativeDebugData &message) { QMutexLocker locker(&m_mutex); - if (m_deferredSend - || (QThread::currentThread() != QCoreApplication::instance()->thread())) - m_data.append(message); - else - sendMessage(message.toByteArray()); + m_data.append(message); } /* @@ -247,19 +243,17 @@ void QDeclarativeDebugTrace::processMessage(const QDeclarativeDebugData &message */ void QDeclarativeDebugTrace::sendMessages() { - if (m_deferredSend) { - QMutexLocker locker(&m_mutex); - //### this is a suboptimal way to send batched messages - for (int i = 0; i < m_data.count(); ++i) - sendMessage(m_data.at(i).toByteArray()); - m_data.clear(); - - //indicate completion - QByteArray data; - QDataStream ds(&data, QIODevice::WriteOnly); - ds << (qint64)-1 << (int)Complete; - sendMessage(data); - } + QMutexLocker locker(&m_mutex); + //### this is a suboptimal way to send batched messages + for (int i = 0; i < m_data.count(); ++i) + sendMessage(m_data.at(i).toByteArray()); + m_data.clear(); + + //indicate completion + QByteArray data; + QDataStream ds(&data, QIODevice::WriteOnly); + ds << (qint64)-1 << (int)Complete; + sendMessage(data); } void QDeclarativeDebugTrace::messageReceived(const QByteArray &message) diff --git a/src/declarative/debugger/qdeclarativedebugtrace_p.h b/src/declarative/debugger/qdeclarativedebugtrace_p.h index 51f3e60b45..07af0004dd 100644 --- a/src/declarative/debugger/qdeclarativedebugtrace_p.h +++ b/src/declarative/debugger/qdeclarativedebugtrace_p.h @@ -147,7 +147,6 @@ private: void sendMessages(); QElapsedTimer m_timer; bool m_enabled; - bool m_deferredSend; bool m_messageReceived; QVector<QDeclarativeDebugData> m_data; QMutex m_mutex; |