From: Václav Slavík <vslavik@fastmail.fm>
Date: Sun, 3 Jun 2001 09:43:36 +0000 (+0000)
Subject: fix for wxFrame's last focus bug (finally?)
X-Git-Url: https://git.saurik.com/wxWidgets.git/commitdiff_plain/cc0c7cd8a5c94494bd9326520960e915dea4cd5e

fix for wxFrame's last focus bug (finally?)


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

diff --git a/src/msw/window.cpp b/src/msw/window.cpp
index fb7bb67368..7b171f9f7c 100644
--- a/src/msw/window.cpp
+++ b/src/msw/window.cpp
@@ -297,15 +297,18 @@ wxWindow::~wxWindow()
         {
             if ( frame->GetLastFocus() == this )
                 frame->SetLastFocus((wxWindow*)NULL);
-            //break;
+            break;
         }
     }
 
+    // VS: destroy children first and _then_ detach *this from its parent.
+    //     If we'd do it the other way around, children wouldn't be able
+    //     find their parent frame (see above).
+    DestroyChildren();
+
     if ( m_parent )
         m_parent->RemoveChild(this);
 
-    DestroyChildren();
-
     if ( m_hWnd )
     {
         // VZ: test temp removed to understand what really happens here