#175 application_launched signal incorrectly fired multiple times

koekeishiya · Jul 30, 2019 · 2908ea0 · 2908ea0
1 parent a02ca96
commit 2908ea0
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -25,6 +25,7 @@ This project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.htm
 - Revert change that made the status bar draw above other windows because of compatibility with "windowed fullscreen" applications [#170](https://github.com/koekeishiya/yabai/issues/170)
 - Warping a window should respect insert direction in scenarios where the default warp is equal to a swap operation [#146](https://github.com/koekeishiya/yabai/issues/146)
 - Exiting mission-control will invalidate the region assigned to Spaces/Views as because a Space may have been dragged to a different monitor [#](https://github.com/koekeishiya/yabai/issues/118)
+- Application_Launched signal incorrectly fired multiple times due to accessibility retries [#175](https://github.com/koekeishiya/yabai/issues/175)
 
 ## [1.1.2] - 2019-07-15
 ### Changed

diff --git a/src/event.c b/src/event.c
@@ -251,6 +251,8 @@ static EVENT_CALLBACK(EVENT_HANDLER_APPLICATION_LAUNCHED)
             event_loop_post(&g_event_loop, event);
             window_manager_remove_lost_front_switched_event(&g_window_manager, process->pid);
         }
+
+        return EVENT_SUCCESS;
     } else {
         bool retry_ax = application->retry;
         application_unobserve(application);
@@ -264,9 +266,9 @@ static EVENT_CALLBACK(EVENT_HANDLER_APPLICATION_LAUNCHED)
                 event_loop_post(&g_event_loop, event);
             });
         }
-    }
 
-    return EVENT_SUCCESS;
+        return EVENT_FAILURE;
+    }
 }
 
 static EVENT_CALLBACK(EVENT_HANDLER_APPLICATION_TERMINATED)