Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

[update] change 'usleep()' to 'add Runloop Source ' #28

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

Conversation

birdmichael
Copy link
Member

@birdmichael birdmichael commented Jul 29, 2019

CheckList

Thanks for considering to this repository. Before you submit your issue, please confirm these boxes are checked.

  • I have read 《[EN]Style guide for Swift repo [CN]Swift项目代码规范、规约选型》.
  • 我遵守一下 Merge 规则:邀请群组中任意一人进行review,即可合并。让 reviewer 在 PR 所在页面写下review意见表示通过,即可合并。最简单写一句 LGTM 也可以。不按照本规则执行,自己 PR 自己 Merge 并引入问题,会被收回 Merge 权限。

My issue:


What I have done:

i think should use an prefer other techniques like RunLoop scheduling.

https://developer.apple.com/library/mac/#documentation/Cocoa/Reference/Foundation/Classes/nsrunloop_Class/Reference/Reference.html

using usleep() will arise a lot of problems related to multithreading
using usleep() is not a good alternative, because it may block the GUI

@ChenYilong ChenYilong mentioned this pull request Jul 29, 2019
2 tasks
@ChenYilong
Copy link
Member

Reference:

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

Successfully merging this pull request may close these issues.

2 participants