crash in some very special case when arrows were used to move around fixed

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

diff --git a/src/generic/treectrl.cpp b/src/generic/treectrl.cpp
index b2ebd5e3a035e19cbf22b0440c22f46cb0c5eae3..d9cb6dbb249bcdf3ce9e237344899ec003635dc9 100644 (file)
--- a/src/generic/treectrl.cpp
+++ b/src/generic/treectrl.cpp
@@ -1307,14 +1307,15 @@ void wxTreeCtrl::OnChar( wxKeyEvent &event )
                 }
                 if (prev)
                 {
                 }
                 if (prev)
                 {

-                    while (IsExpanded(prev))
+                    while ( IsExpanded(prev) && HasChildren(prev) )

                     {
                     {

-                        int c = (int)GetChildrenCount( prev, FALSE );
-                        long cockie = 0;
-                        prev = GetFirstChild( prev, cockie );
-                        for (int i = 0; i < c-1; i++)
-                            prev = GetNextSibling( prev );
+                        wxTreeItemId child = GetLastChild(prev);
+                        if ( child )
+                        {
+                            prev = child;
+                        }

                     }
                     }

+

                     SelectItem( prev );
                     EnsureVisible( prev );
                 }
                     SelectItem( prev );
                     EnsureVisible( prev );
                 }


@@ -1341,7 +1342,7 @@ void wxTreeCtrl::OnChar( wxKeyEvent &event )
 
         case WXK_DOWN:
             {
 
         case WXK_DOWN:
             {

-                if (IsExpanded(m_current))
+                if (IsExpanded(m_current) && HasChildren(m_current))

                 {
                     long cookie = 0;
                     wxTreeItemId child = GetFirstChild( m_current, cookie );
                 {
                     long cookie = 0;
                     wxTreeItemId child = GetFirstChild( m_current, cookie );