Replacing four process.nextTick calls with four setImmediate calls. S… #806
Conversation
…hould get rid of the node warnings that are blowing up my call stack.
|
What are the problems that this addresses? |
|
This pull request is 2 years old. @kibertoad https://nodejs.org/en/docs/guides/event-loop-timers-and-nexttick/ is the official explanation of whats going on, but some important bits are below:
And in summary:
They give a few examples of why someone may want to explicitly use nextTick.. but ultimately you're breaking out of the event loop in an unexpected way. |
Response to #715
The process.nextTick deprecation warnings are causing problems on one of my servers. This appears to resolve those problems. Two tests were broken pre-commit, and those two tests are still broken, but no more have broken.