Skip to content

Allow to call FSEventsEmitter.stop() twice without error#466

Merged
BoboTiG merged 3 commits into
gorakhargosh:masterfrom
LKleinNux:fix-fseventsemitter-stop
Sep 19, 2018
Merged

Allow to call FSEventsEmitter.stop() twice without error#466
BoboTiG merged 3 commits into
gorakhargosh:masterfrom
LKleinNux:fix-fseventsemitter-stop

Conversation

@LKleinNux

Copy link
Copy Markdown
Contributor

Calling .stop() on an already stopped emitter will do nothing instead of raising an error on a null pointer.

@BoboTiG

BoboTiG commented Sep 10, 2018

Copy link
Copy Markdown
Collaborator

Thanks! Could you fix tests?

@LKleinNux
LKleinNux force-pushed the fix-fseventsemitter-stop branch from d85459d to 4cfb421 Compare September 19, 2018 12:30
@BoboTiG

BoboTiG commented Sep 19, 2018

Copy link
Copy Markdown
Collaborator

Tests are OK, I am merging. Thanks @LKleinNux :)

@BoboTiG
BoboTiG merged commit e2730fd into gorakhargosh:master Sep 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants