Amebis/wxWidgets
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

MouseEvent crash fix on wxWindow Destroy

Browse Source 
If the event handler destroys the wxWindow, then the subsequent call to SetupCursor(event) will miserably fail .
Moved setting the cursor before the event is handled.
This commit is contained in:
tommash
2017-12-15 15:01:26 +01:00
committed by GitHub
parent 276d96b583
commit 3de6d1eb17
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/osx/cocoa/window.mm
View File

@@ -3579,11 +3579,11 @@ bool wxWidgetCocoaImpl::DoHandleKeyEvent(NSEvent *event)
bool wxWidgetCocoaImpl::DoHandleMouseEvent(NSEvent *event) bool wxWidgetCocoaImpl::DoHandleMouseEvent(NSEvent *event)
{ {
(void)SetupCursor(event);
wxMouseEvent wxevent(wxEVT_LEFT_DOWN); wxMouseEvent wxevent(wxEVT_LEFT_DOWN);
SetupMouseEvent(wxevent , event) ; SetupMouseEvent(wxevent , event) ;
bool result = GetWXPeer()->HandleWindowEvent(wxevent); bool result = GetWXPeer()->HandleWindowEvent(wxevent);
(void)SetupCursor(event);
return result; return result;
} }