i2p/haiku-beta6
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

TimeView: fix direct messaging to another window

Browse Source 
The window may be gone (if its B_QUIT_REQUESTED was called before the
main Deskbar window one), so we can't safely call Quit() on it. Instead,
just send a B_QUIT_REQUESTED through the BMessenger, which is a bit
safer.

Should fix #16026.

Change-Id: I4c99c3a7f994044c3c3f0c08b35ae451c3ff9ea7
Reviewed-on: https://review.haiku-os.org/c/haiku/+/2721
Reviewed-by: waddlesplash <[email protected]>
This commit is contained in:
Adrien Destugues
2020-05-17 18:27:13 +00:00
committed by waddlesplash
parent d9ef4f90bb
commit 54bff8ec82
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/apps/deskbar/TimeView.cpp
+2 -2
View File
@@ -110,8 +110,8 @@ TTimeView::TTimeView(BMessage* data)
TTimeView::~TTimeView() TTimeView::~TTimeView()
{ {
if (fCalendarWindow != NULL) if (fCalendarWindowMessenger.IsValid())
fCalendarWindow->Quit(); fCalendarWindowMessenger.PostMessage(B_QUIT_REQUESTED);
delete fTimeFormat; delete fTimeFormat;
delete fDateFormat; delete fDateFormat;