comparison historywidget.cpp @ 138:a7dbc8e5b69d

* Minor adjustment to scene lifecycle, delete old scene before creating new one to use less memory &c

 }
 
 void HistoryWidget::addChangesets(Changesets csets)
 {
     m_newIds.clear();
+
+    if (csets.empty()) return;
+
     foreach (Changeset *cs, csets) {
         m_newIds.insert(cs->id());
     }
 
+    DEBUG << "addChangesets: " << csets.size() << " new changesets" << endl;
+
     csets << m_changesets;
     m_changesets = csets;
 }
 
 void HistoryWidget::layoutAll()
 {
     setChangesetParents();
 
     ChangesetScene *scene = new ChangesetScene();
     ChangesetItem *tipItem = 0;
+
+    QGraphicsScene *oldScene = m_panned->scene();
+
+    // detach m_uncommitted from old scene so it doesn't get deleted
+    if (oldScene && (m_uncommitted->scene() == oldScene)) {
+        oldScene->removeItem(m_uncommitted);
+    }
+
+    m_panned->setScene(0);
+    m_panner->setScene(0);
+
+    delete oldScene;
 
     if (!m_changesets.empty()) {
 	Grapher g(scene);
 	try {
 	    g.layout(m_changesets);

 	tipItem = g.getItemFor(m_changesets[0]);
         DEBUG << "tipItem is " << tipItem << " for tip changeset " 
               << m_changesets[0]->id() << endl;
     }
 
-    QGraphicsScene *oldScene = m_panned->scene();
-
-    // detach m_uncommitted from old scene so it doesn't get deleted
-    if (oldScene && (m_uncommitted->scene() == oldScene)) {
-        oldScene->removeItem(m_uncommitted);
-    }
     if (m_uncommittedVisible) {
         scene->addItem(m_uncommitted);
     }
 
     m_panned->setScene(scene);
     m_panner->setScene(scene);
-
-    if (oldScene) delete oldScene;
 
     updateNewAndCurrentItems();
 
     if (m_uncommittedVisible) {
         m_uncommitted->ensureVisible();