Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

MultiThreading #14

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

MultiThreading #14

wants to merge 2 commits into from

Conversation

macintosh264
Copy link

Added multithreading to the table so you don't hang up the main thread with network requests.

@zadr
Copy link
Contributor

zadr commented Jan 29, 2012

performSelector:withObject:afterDelay: is non-blocking and will defer to the runloop. I suspect (but haven't bothered benchmarking) that spawning a new thread, and then dispatching back to the main runloop is actually more expensive than letting the runloop handle things outright.

@macintosh264
Copy link
Author

It is useful when executing network requests, and thus is better than letting it run on the main runloop.

@monzonj
Copy link

monzonj commented May 11, 2012

Actually, the refresh method should be replaced by a delegate that performs whatever request is needed to update this table view. This controller is generic and should be oblivious about implementations details

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants