From f60d8e0b59a6181cac608787b4aab21110cb6f0a Mon Sep 17 00:00:00 2001
From: Stefano Ceccherini <stefano.ceccherini@gmail.com>
Date: Wed, 23 Aug 2006 06:42:09 +0000
Subject: [PATCH] Replaced Unlock() with UnlockFully() in BWindow's destructor,
 fixes bug 406

git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@18579 a95241bf-73f2-0310-859d-f6bbb57e9c96
---
 src/kits/interface/Window.cpp | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/kits/interface/Window.cpp b/src/kits/interface/Window.cpp
index 81bace21d9..b356e6f282 100644
--- a/src/kits/interface/Window.cpp
+++ b/src/kits/interface/Window.cpp
@@ -357,7 +357,10 @@ BWindow::~BWindow()
 	// The BWindow is locked when the destructor is called,
 	// we need to unlock because the menubar thread tries
 	// to post a message, which will deadlock otherwise.
-	Unlock();
+	// TODO: I replaced Unlock() with UnlockFully() because the window
+	// was kept locked after that in case it was closed using ALT-W.
+	// There might be an extra Lock() somewhere in the quitting path...
+	UnlockFully();
 
 	// Wait if a menu is still tracking
 	if (fMenuSem > 0) {