Skip to content
This repository has been archived by the owner on May 4, 2018. It is now read-only.

fsevents: use shared FSEventStream #894

Closed
wants to merge 3 commits into from
Closed

fsevents: use shared FSEventStream #894

wants to merge 3 commits into from

Commits on Aug 21, 2013

  1. fsevents: use shared FSEventStream 

    It seems that number of simultaneously opened FSEventStreams is
limited on OSX (i.e. you can have only fixed number of them on
one running system), getting past through this limit will cause
`FSEventStreamCreate` to return false and write following message
to stderr:

    (CarbonCore.framework) FSEventStreamStart: register_with_server:
    ERROR: f2d_register_rpc() => (null) (-21)

To prevent this, we must use only one shared FSEventStream with a
paths for all uv_fsevent_t handles, and then filter out events for
each handle using this paths again.

See nodejs/node-v0.x-archive#5463
    @indutny
    indutny committed Aug 21, 2013
    Configuration menu
    Copy the full SHA
    c690f00 View commit details
    Browse the repository at this point in the history

  2. fsevents: rename WALK => PROCESS 

    It isn't really walking through events, because it changes (resets)
them.

Per @bnoordhuis and @tjfontaine suggestion.
    @indutny
    indutny committed Aug 21, 2013
    Configuration menu
    Copy the full SHA
    3261ec1 View commit details
    Browse the repository at this point in the history

  3. fixes

    @indutny
    indutny committed Aug 21, 2013
    Configuration menu
    Copy the full SHA
    df6309a View commit details
    Browse the repository at this point in the history