In some circumstances we could call wxApp::OnExit() before destroying the application windows, which violated the documented (and reasonably expected) behaviour.
Fix this by adding a wrapper CallOnExit() function which destroys all windows before calling OnExit() and, for good measure, then does it again in case the user-overridden version of this function created more windows when it was called.
This also has a nice side effect of freeing the user code from the need to call the base OnExit(), as it is now trivial and doesn't do anything any more.
Closes #26172.
https://github.com/wxWidgets/wxWidgets/pull/26189
(6 files)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.![]()
@vadz pushed 1 commit.
—
View it on GitHub or unsubscribe.
You are receiving this because you are subscribed to this thread.![]()
@vadz pushed 1 commit.
—
View it on GitHub or unsubscribe.
You are receiving this because you are subscribed to this thread.![]()
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.![]()
Cherry picked in the commit referenced above.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.![]()