Skip to content
This repository has been archived by the owner on Jul 20, 2023. It is now read-only.

Moved Creation of AVURLAsset to background thread. #67

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

joelmbell
Copy link

@joelmbell joelmbell commented Mar 17, 2017

Addresses Issue #57

AVURLAsset has some undocumented behavior where passing specific url's
to it with the AVAsset(url:) initializer will cause main thread stalls.

I simply moved the creation of this object onto a background thread to
avoid any of these potential stalls happening on the main thread.

AVURLAsset has some undocumented behavior where passing specific url's
to it with the AVAsset(url:) initializer will cause main thread stalls.

I simply moved the creation of this object onto a background thread to
avoid any of these potential stalls happening on the main thread.
@joelmbell joelmbell changed the title Moved Creation of AVURLAsset to background thread. Addresses Issue #57 Moved Creation of AVURLAsset to background thread. Mar 17, 2017
@joelmbell
Copy link
Author

Hmm, not sure why the build failed. All tests pass locally.

Please let me know if there is anything I should be doing differently to get this to pass. I'd happily update the branch.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant