From 0b93babdc00c00a8ee121ed053365bc9a37f725b Mon Sep 17 00:00:00 2001
From: =?utf8?q?V=C3=A1clav=20Slav=C3=ADk?= <vslavik@fastmail.fm>
Date: Fri, 19 Aug 2011 18:25:28 +0000
Subject: [PATCH] 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
---
 src/generic/datavgen.cpp | 39 ++++++++++++++++++++-------------------
 1 file changed, 20 insertions(+), 19 deletions(-)

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;
-- 
2.47.2