GeezerVison

The World Wide Web, restored to 800 pixels of glory.

GeezerVison is showing github.com — process: do not lazily load AsyncResource · nodejs/node@e9110d5

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

lib/internal/process
- task_queues.js
test/parallel
- test-bootstrap-modules.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

`‎lib/internal/process/task_queues.js‎`

Copy file name to clipboardExpand all lines: lib/internal/process/task_queues.js

+8-11Lines changed: 8 additions & 11 deletions

Original file line number	Diff line number	Diff line change
`@@ -39,6 +39,8 @@ const {`
`39`	`39`	`} = require('internal/errors').codes;`
`40`	`40`	`const FixedQueue = require('internal/fixed_queue');`
`41`	`41`
	`42`	`+const { AsyncResource } = require('async_hooks');`
	`43`	`+`
`42`	`44`	`// Must match Environment::TickInfo::Fields in src/env.h.`
`43`	`45`	`const kHasTickScheduled = 0;`
`44`	`46`
`@@ -132,16 +134,6 @@ function nextTick(callback) {`
`132`	`134`	`queue.push(tickObject);`
`133`	`135`	`}`
`134`	`136`
`135`		`-let AsyncResource;`
`136`		`-const defaultMicrotaskResourceOpts = { requireManualDestroy: true };`
`137`		`-function createMicrotaskResource() {`
`138`		`- // Lazy load the async_hooks module`
`139`		`- if (AsyncResource === undefined) {`
`140`		`- AsyncResource = require('async_hooks').AsyncResource;`
`141`		`- }`
`142`		`- return new AsyncResource('Microtask', defaultMicrotaskResourceOpts);`
`143`		`-}`
`144`		`-`
`145`	`137`	`function runMicrotask() {`
`146`	`138`	`this.runInAsyncScope(() => {`
`147`	`139`	`const callback = this.callback;`
`@@ -153,12 +145,17 @@ function runMicrotask() {`
`153`	`145`	`});`
`154`	`146`	`}`
`155`	`147`
	`148`	`+const defaultMicrotaskResourceOpts = { requireManualDestroy: true };`
	`149`	`+`
`156`	`150`	`function queueMicrotask(callback) {`
`157`	`151`	`if (typeof callback !== 'function') {`
`158`	`152`	`throw new ERR_INVALID_ARG_TYPE('callback', 'function', callback);`
`159`	`153`	`}`
`160`	`154`
`161`		`- const asyncResource = createMicrotaskResource();`
	`155`	`+ const asyncResource = new AsyncResource(`
	`156`	`+ 'Microtask',`
	`157`	`+ defaultMicrotaskResourceOpts`
	`158`	`+ );`
`162`	`159`	`asyncResource.callback = callback;`
`163`	`160`
`164`	`161`	`enqueueMicrotask(FunctionPrototypeBind(runMicrotask, asyncResource));`

Collapse file

`‎test/parallel/test-bootstrap-modules.js‎`

Copy file name to clipboardExpand all lines: test/parallel/test-bootstrap-modules.js

+1Lines changed: 1 addition & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -31,6 +31,7 @@ const expectedModules = new Set([`
`31`	`31`	`'Internal Binding types',`
`32`	`32`	`'Internal Binding url',`
`33`	`33`	`'Internal Binding util',`
	`34`	`+ 'NativeModule async_hooks',`
`34`	`35`	`'NativeModule buffer',`
`35`	`36`	`'NativeModule events',`
`36`	`37`	`'NativeModule fs',`

0 commit comments

Comments

0 (0)

You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.