[vlc-commits] Revert "macosx: Spin event loop to process pending selectors on main thread"

David Fuhrmann git at videolan.org
Sun Mar 18 19:02:16 CET 2018


vlc | branch: master | David Fuhrmann <dfuhrmann at videolan.org> | Sun Mar 18 18:28:37 2018 +0100| [bbae6326df1dc79b9bbfde520767b0f3b10d7c56] | committer: David Fuhrmann

Revert "macosx: Spin event loop to process pending selectors on main thread"

This caused too many unrelated crashes / problems.

This partly reverts commit 5d0d2b0d533df40c6e78460d2a320997e7e3ee34.

close #19983

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=bbae6326df1dc79b9bbfde520767b0f3b10d7c56
---

 modules/gui/macosx/VLCMain.m | 10 ----------
 1 file changed, 10 deletions(-)

diff --git a/modules/gui/macosx/VLCMain.m b/modules/gui/macosx/VLCMain.m
index f78e694119..033d555294 100644
--- a/modules/gui/macosx/VLCMain.m
+++ b/modules/gui/macosx/VLCMain.m
@@ -115,16 +115,6 @@ void CloseIntf (vlc_object_t *p_this)
         [[VLCMain sharedInstance] applicationWillTerminate:nil];
         [VLCMain killInstance];
 
-        /*
-         * Spinning the event loop here is important to help cleaning up all objects which should be
-         * destroyed here. Its possible that main thread selectors (which hold a strong reference
-         * to the target object), are still in the queue (e.g. fired from variable callback).
-         * Thus make sure those are still dispatched and the references to the targets are
-         * cleared, to allow the objects to be released.
-         */
-        msg_Dbg(p_this, "Spin the event loop to clean up the interface");
-        [[NSRunLoop mainRunLoop] runUntilDate:[NSDate date]];
-
         p_interface_thread = nil;
     }
 }



More information about the vlc-commits mailing list