From 0b93babdc00c00a8ee121ed053365bc9a37f725b Mon Sep 17 00:00:00 2001
From: =?UTF-8?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 | 47 ++++++++++++++++++++--------------------
 1 file changed, 24 insertions(+), 23 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;
-    }
-
-    SortPrepare();
-
-    wxDataViewTreeNode * node;
-    node = FindNode(parent);
-
-    if( node == NULL )
-        return false;
-
-    node->SetHasChildren( true );
-
-    if( g_model->IsContainer( item ) )
-    {
-        wxDataViewTreeNode * newnode = new wxDataViewTreeNode( node );
-        newnode->SetItem(item);
-        newnode->SetHasChildren( true );
-        node->AddNode( newnode);
     }
     else
-        node->AddLeaf( item.GetID() );
+    {
+        SortPrepare();
 
-    node->ChangeSubTreeCount(1);
+        wxDataViewTreeNode * node;
+        node = FindNode(parent);
+
+        if( node == NULL )
+            return false;
+
+        node->SetHasChildren( true );
+
+        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->ChangeSubTreeCount(1);
+
+        m_count = -1;
+    }
 
-    m_count = -1;
     UpdateDisplay();
 
     return true;