X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/d285d708a229ece6ad606a460bcbf048c65d252b..3fc6e5fa23909c701654001a895f2ba95751e0af:/src/msw/toplevel.cpp diff --git a/src/msw/toplevel.cpp b/src/msw/toplevel.cpp index 9ce65687c6..1e6fc3de12 100644 --- a/src/msw/toplevel.cpp +++ b/src/msw/toplevel.cpp @@ -454,12 +454,12 @@ wxTopLevelWindowMSW::~wxTopLevelWindowMSW() if ( this == ms_hiddenParent ) { // stop [infinite] recursion which would otherwise happen when we do - // "delete ms_hiddenParent" below + // "delete ms_hiddenParent" below -- and we're not interested in doing + // anything of the rest below for that window because the rest of + // wxWindows doesn't even know about it return; } - wxTopLevelWindows.DeleteObject(this); - if ( wxModelessWindows.Find(this) ) wxModelessWindows.DeleteObject(this); @@ -476,21 +476,15 @@ wxTopLevelWindowMSW::~wxTopLevelWindowMSW() } } - // If this is the last top-level window, exit. - if ( wxTheApp && (wxTopLevelWindows.Number() == 0) ) + // if this is the last top-level window, we're going to exit and we should + // delete ms_hiddenParent now to avoid leaking it + if ( IsLastBeforeExit() ) { if ( ms_hiddenParent ) { delete ms_hiddenParent; ms_hiddenParent = NULL; } - - wxTheApp->SetTopWindow(NULL); - - if ( wxTheApp->GetExitOnFrameDelete() ) - { - ::PostQuitMessage(0); - } } }