From fc7e282f1c99ed6a00051c67fd7a01d44e57b8ed Mon Sep 17 00:00:00 2001 From: hatimhtm <106043141+hatimhtm@users.noreply.github.com> Date: Sat, 21 Mar 2026 10:49:44 +0000 Subject: [PATCH] Optimize O(N) appID string check to an O(1) Set lookup Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com> --- Click2Minimize/AppDelegate.swift | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Click2Minimize/AppDelegate.swift b/Click2Minimize/AppDelegate.swift index 55a4109..5dc29cd 100644 --- a/Click2Minimize/AppDelegate.swift +++ b/Click2Minimize/AppDelegate.swift @@ -41,6 +41,7 @@ class AppDelegate: NSObject, NSApplicationDelegate { var mainWindow: NSWindow? var cancellables = Set() var dockItems: [DockItem] = [] // Global variable to hold dock item rectangles + private static let ignoredAppIDs: Set = ["Launchpad", "Trash", "Downloads"] private var isClickToMinimizeEnabled: Bool = { // Set isClickToMinimizeEnabled to true if not found if UserDefaults.standard.object(forKey: "ClickToMinimizeEnabled") == nil { UserDefaults.standard.set(true, forKey: "ClickToMinimizeEnabled") // Set default value @@ -183,7 +184,7 @@ class AppDelegate: NSObject, NSApplicationDelegate { if dockItem.rect.contains(mouseLocation) { // Log the mouse location and app name print("Mouse Location: \(mouseLocation), App Name: \(dockItem.appID)") - if "Launchpad||Trash||Downloads".contains(dockItem.appID) { + if AppDelegate.ignoredAppIDs.contains(dockItem.appID) { // these are not working for sure return Unmanaged.passUnretained(event) }