From cd03d7c12004ef95ed8e942b8affc8cced7be1c6 Mon Sep 17 00:00:00 2001 From: John Scipione Date: Fri, 21 Dec 2012 16:42:37 -0500 Subject: [PATCH] Remove locking from TrackerSettingsWindow::QuitRequested() Like Deskbar it isn't needed here either since the window is already locked. The return value seems to be totally ignored, so, just leave as is. --- src/kits/tracker/TrackerSettingsWindow.cpp | 11 +---------- 1 file changed, 1 insertion(+), 10 deletions(-) diff --git a/src/kits/tracker/TrackerSettingsWindow.cpp b/src/kits/tracker/TrackerSettingsWindow.cpp index 552f073618..4be73c1bc3 100644 --- a/src/kits/tracker/TrackerSettingsWindow.cpp +++ b/src/kits/tracker/TrackerSettingsWindow.cpp @@ -135,19 +135,10 @@ TrackerSettingsWindow::TrackerSettingsWindow() bool TrackerSettingsWindow::QuitRequested() { - bool isHidden = false; - - if (Lock()) { - isHidden = IsHidden(); - Unlock(); - } else - return true; - - if (isHidden) + if (IsHidden()) return true; Hide(); - return false; }