]> git.cworth.org Git - apitrace/commitdiff
Fix minor errors in the apitracemodel
authorZack Rusin <zack@kde.org>
Thu, 21 Apr 2011 03:58:52 +0000 (23:58 -0400)
committerZack Rusin <zack@kde.org>
Thu, 21 Apr 2011 03:58:52 +0000 (23:58 -0400)
gui/apitrace.cpp
gui/apitrace.h
gui/apitracemodel.cpp
gui/apitracemodel.h

index eb65ba8754eb5d359f06a5d4997234bba02966ef..4c96c717c5643649470d5ff5914ab0090c2533fe 100644 (file)
@@ -146,6 +146,9 @@ void ApiTrace::addFrames(const QList<ApiTraceFrame*> &frames)
 {
     int currentFrames = m_frames.count();
     int numNewFrames = frames.count();
+
+    emit beginAddingFrames(currentFrames, numNewFrames);
+
     m_frames += frames;
 
     int currentCalls = m_calls.count();
@@ -156,7 +159,7 @@ void ApiTrace::addFrames(const QList<ApiTraceFrame*> &frames)
         m_calls += frame->calls;
     }
 
-    emit framesAdded(currentFrames, numNewFrames);
+    emit endAddingFrames();
     emit callsAdded(currentCalls, numNewCalls);
 }
 
@@ -165,6 +168,8 @@ void ApiTrace::detectFrames()
     if (m_calls.isEmpty())
         return;
 
+    emit beginAddingFrames(0, m_frames.count());
+
     ApiTraceFrame *currentFrame = 0;
     foreach(ApiTraceCall *apiCall, m_calls) {
         if (!currentFrame) {
@@ -187,7 +192,7 @@ void ApiTrace::detectFrames()
         m_frames.append(currentFrame);
         currentFrame = 0;
     }
-    emit framesAdded(0, m_frames.count());
+    emit endAddingFrames();
 }
 
 ApiTraceCall * ApiTrace::callWithIndex(int idx) const
index d0b76fa6b5dc0c1894730a9e86b4df7ffcf8bc4d..d52c669928aeac70654bf3c9a48be89fbaa04698 100644 (file)
@@ -68,7 +68,8 @@ signals:
     void startedSaving();
     void saved();
 
-    void framesAdded(int oldCount, int numAdded);
+    void beginAddingFrames(int oldCount, int numAdded);
+    void endAddingFrames();
     void callsAdded(int oldCount, int numAdded);
 
 private slots:
index 951d6ac085c006dc9eff305e9f8b34e00020c66c..debf5c5712698470c27a6ea024b96b4dd2b211ee 100644 (file)
@@ -98,7 +98,7 @@ QVariant ApiTraceModel::headerData(int section, Qt::Orientation orientation,
 QModelIndex ApiTraceModel::index(int row, int column,
                                  const QModelIndex &parent) const
 {
-    if (parent.isValid() && parent.column() != 0)
+    if ((parent.isValid() && parent.column() != 0) || column != 0)
         return QModelIndex();
 
     ApiTraceEvent *event = item(parent);
@@ -204,8 +204,10 @@ void ApiTraceModel::setApiTrace(ApiTrace *trace)
             this, SLOT(invalidateFrames()));
     connect(m_trace, SIGNAL(framesInvalidated()),
             this, SLOT(invalidateFrames()));
-    connect(m_trace, SIGNAL(framesAdded(int, int)),
-            this, SLOT(appendFrames(int, int)));
+    connect(m_trace, SIGNAL(beginAddingFrames(int, int)),
+            this, SLOT(beginAddingFrames(int, int)));
+    connect(m_trace, SIGNAL(endAddingFrames()),
+            this, SLOT(endAddingFrames()));
     connect(m_trace, SIGNAL(changed(ApiTraceCall*)),
             this, SLOT(callChanged(ApiTraceCall*)));
 }
@@ -221,11 +223,10 @@ void ApiTraceModel::invalidateFrames()
     endResetModel();
 }
 
-void ApiTraceModel::appendFrames(int oldCount, int numAdded)
+void ApiTraceModel::beginAddingFrames(int oldCount, int numAdded)
 {
     beginInsertRows(QModelIndex(), oldCount,
                     oldCount + numAdded - 1);
-    endInsertRows();
 }
 
 ApiTraceEvent * ApiTraceModel::item(const QModelIndex &index) const
@@ -299,4 +300,9 @@ void ApiTraceModel::callChanged(ApiTraceCall *call)
     emit dataChanged(index, index);
 }
 
+void ApiTraceModel::endAddingFrames()
+{
+    endInsertRows();
+}
+
 #include "apitracemodel.moc"
index 752d7b654436a92c2e920a60de51a7e4438472b0..2cdd7d0bfe6fd9e1cfcbd83f25bfe4a018aab262 100644 (file)
@@ -49,7 +49,8 @@ public:
 
 private slots:
     void invalidateFrames();
-    void appendFrames(int oldCount, int numAdded);
+    void beginAddingFrames(int oldCount, int numAdded);
+    void endAddingFrames();
     void callChanged(ApiTraceCall *call);
 
 private: