diff options
author | Joyee Cheung <joyeec9h3@gmail.com> | 2018-12-19 18:44:14 +0800 |
---|---|---|
committer | Joyee Cheung <joyeec9h3@gmail.com> | 2018-12-24 07:57:15 +0800 |
commit | 457603e96194b4858ad715f9faacb3ad7fec7f35 (patch) | |
tree | c9d496b06366721b7453b38c176b1cdbaa39fbe9 /src/node_binding.cc | |
parent | e830e2742cedceb7fc89de4910b2c1a9536a1638 (diff) | |
download | node-new-457603e96194b4858ad715f9faacb3ad7fec7f35.tar.gz |
src: move process.nextTick and promise setup into node_task_queue.cc
This patch:
- Moves the process.nextTick and promise setup C++ code into
node_task_queue.cc which is exposed as
`internalBinding('task_queue')`
- Makes `lib/internal/process/promises.js` and
`lib/internal/process/next_tick.js` as side-effect-free
as possible
- Removes the bootstrapper object being passed into
`bootstrap/node.js`, let `next_tick.js` and `promises.js`
load whatever they need from `internalBinding('task_queue')`
instead.
- Rename `process._tickCallback` to `runNextTicks` internally
for clarity but still expose it as `process._tickCallback`.
PR-URL: https://github.com/nodejs/node/pull/25163
Refs: https://github.com/nodejs/node/issues/24961
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Diffstat (limited to 'src/node_binding.cc')
-rw-r--r-- | src/node_binding.cc | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/node_binding.cc b/src/node_binding.cc index 46ae5f6840..cf47b3058d 100644 --- a/src/node_binding.cc +++ b/src/node_binding.cc @@ -52,6 +52,7 @@ V(stream_wrap) \ V(string_decoder) \ V(symbols) \ + V(task_queue) \ V(tcp_wrap) \ V(timers) \ V(trace_events) \ |