Skip to content
Snippets Groups Projects
Commit 20e5fe4a authored by Alexandre Janniaux's avatar Alexandre Janniaux
Browse files

VLCVideoUIView: remove racy assert

_enabled is set asynchronously in the main thread whereas
detachFromParent is called from the thread closing the window, which is
usually different from the main thread. Checking _enabled here is racy
and could lead to triggering the assertion.
parent dcb8bc8a
No related branches found
No related tags found
No related merge requests found
......@@ -171,7 +171,6 @@
/* The UIView must not be attached before releasing. Disable() is doing
* exactly this asynchronously in the main thread so ensure it was called
* here before detaching from the parent. */
assert(!_enabled);
_wnd = NULL;
vlc_mutex_unlock(&_mutex);
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment