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 /node.gyp | |
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 'node.gyp')
-rw-r--r-- | node.gyp | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -325,7 +325,6 @@ 'sources': [ 'src/async_wrap.cc', - 'src/bootstrapper.cc', 'src/callback_scope.cc', 'src/cares_wrap.cc', 'src/connect_wrap.cc', @@ -372,6 +371,7 @@ 'src/node_serdes.cc', 'src/node_stat_watcher.cc', 'src/node_symbols.cc', + 'src/node_task_queue.cc', 'src/node_trace_events.cc', 'src/node_types.cc', 'src/node_url.cc', |