Skip to content

[client] land new changes of new-sync for osx #741

Merged
merged 1 commit into from Jul 25, 2014

3 participants

@Chilledheart
  • look for future review
@killing killing merged commit 24a679f into haiwen:master Jul 25, 2014

1 check passed

Details continuous-integration/travis-ci The Travis CI build passed
@Chilledheart

Thanks for merge!

@Chilledheart Chilledheart deleted the Chilledheart:new-sync-osx branch Jul 25, 2014
@poetwang
haiwen member

should kFSEventStreamEventFlagMustScanSubDirs be handled here?

It is a unusual flag (not so common used), and i believe you'd better have a try yourself. Although i wrote these code, i have nothing to do with these code now because patches have been not accepted since a long time ago. Besides, i thinks it will come with a lot of new events to be handled.

@poetwang
haiwen member

the argument info->worktree seems redundant here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.