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.

Sign up for GitHub

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

Jump to bottom

v0.11.x process.nextTick compatibility #5

Closed
fivdi opened this issue Jan 2, 2014 · 1 comment
Closed

v0.11.x process.nextTick compatibility #5

fivdi opened this issue Jan 2, 2014 · 1 comment

Comments

@fivdi
Copy link
Owner

fivdi commented Jan 2, 2014

The semantics of process.nextTick have changed in Node v0.11.x. Recursively setting nextTick callbacks will block any I/O from happening. In Node v0.10.x this is not the case as nextTick defers to allow some I/O to be done every so often.

See the note at the end of the v0.11.10 process.nextTick() documentation. See also v0.10.24 process.maxTickDepth. maxTickDepth is no longer available in Node v0.11.10.

lcd relies on the v0.10.x semantics here. Replacing process.nextTick(...) with setImmediate(...) should resolve the issue.
@fivdi
Copy link
Owner Author

fivdi commented Mar 28, 2014

Fixed with 3a615ae in v0.2.1

@fivdi fivdi closed this as completed Mar 28, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@fivdi