From: Václav Slavík <vslavik@fastmail.fm>
Date: Fri, 19 Aug 2011 18:25:28 +0000 (+0000)
Subject: Reorganize wxDataViewMainWindow::ItemAdded() for better readability.
X-Git-Url: https://git.saurik.com/wxWidgets.git/commitdiff_plain/0b93babdc00c00a8ee121ed053365bc9a37f725b?hp=409fe759e1b605514eabcddfb546dc876b9a9161

Reorganize wxDataViewMainWindow::ItemAdded() for better readability.

Reuse common code. No real changes.

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@68792 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
---

diff --git a/src/generic/datavgen.cpp b/src/generic/datavgen.cpp
index d2f8fe01bf..233b83f5b3 100644
--- a/src/generic/datavgen.cpp
+++ b/src/generic/datavgen.cpp
@@ -1976,33 +1976,34 @@ bool wxDataViewMainWindow::ItemAdded(const wxDataViewItem & parent, const wxData
         wxDataViewVirtualListModel *list_model =
             (wxDataViewVirtualListModel*) GetOwner()->GetModel();
         m_count = list_model->GetCount();
-        UpdateDisplay();
-        return true;
     }
+    else
+    {
+        SortPrepare();
 
-    SortPrepare();
+        wxDataViewTreeNode * node;
+        node = FindNode(parent);
+
+        if( node == NULL )
+            return false;
 
-    wxDataViewTreeNode * node;
-    node = FindNode(parent);
+        node->SetHasChildren( true );
 
-    if( node == NULL )
-        return false;
+        if( g_model->IsContainer( item ) )
+        {
+            wxDataViewTreeNode * newnode = new wxDataViewTreeNode( node );
+            newnode->SetItem(item);
+            newnode->SetHasChildren( true );
+            node->AddNode( newnode);
+        }
+        else
+            node->AddLeaf( item.GetID() );
 
-    node->SetHasChildren( true );
+        node->ChangeSubTreeCount(1);
 
-    if( g_model->IsContainer( item ) )
-    {
-        wxDataViewTreeNode * newnode = new wxDataViewTreeNode( node );
-        newnode->SetItem(item);
-        newnode->SetHasChildren( true );
-        node->AddNode( newnode);
+        m_count = -1;
     }
-    else
-        node->AddLeaf( item.GetID() );
 
-    node->ChangeSubTreeCount(1);
-
-    m_count = -1;
     UpdateDisplay();
 
     return true;