Commit 5cc2574
src: move async hooks trace events setup to pre_execution.js
Reasons:
- Moves more environment-dependent setup out of bootstrap/node.js
- No async operations should be done before the call to
the setup functions in pre_execution.js so no async hooks should be
triggered before that. Therefore it is safe to delay the setup
until then.
PR-URL: #26062
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>1 parent f0a8166 commit 5cc2574Copy full SHA for 5cc2574
File tree
Expand file treeCollapse file tree
4 files changed
+40
-35
lines changedOpen diff view settings
Filter options
- lib/internal
- bootstrap
- main
Expand file treeCollapse file tree
4 files changed
+40
-35
lines changedOpen diff view settings
Collapse file
lib/internal/bootstrap/node.js
Copy file name to clipboardExpand all lines: lib/internal/bootstrap/node.js-31Lines changed: 0 additions & 31 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
42 | 42 | |
43 | 43 | |
44 | 44 | |
45 | | - |
46 | | - |
47 | 45 | |
48 | 46 | |
49 | 47 | |
| ||
354 | 352 | |
355 | 353 | |
356 | 354 | |
357 | | - |
358 | | - |
359 | | - |
360 | | - |
361 | | - |
362 | | - |
363 | | - |
364 | | - |
365 | | - |
366 | | - |
367 | | - |
368 | | - |
369 | | - |
370 | | - |
371 | | - |
372 | | - |
373 | | - |
374 | | - |
375 | | - |
376 | | - |
377 | | - |
378 | | - |
379 | | - |
380 | | - |
381 | | - |
382 | | - |
383 | | - |
384 | | - |
385 | | - |
386 | 355 | |
387 | 356 | |
388 | 357 | |
|
Collapse file
lib/internal/bootstrap/pre_execution.js
Copy file name to clipboardExpand all lines: lib/internal/bootstrap/pre_execution.js+32-1Lines changed: 32 additions & 1 deletion
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
3 | 3 | |
4 | 4 | |
5 | 5 | |
| 6 | + |
| 7 | + |
6 | 8 | |
7 | 9 | |
8 | 10 | |
| ||
23 | 25 | |
24 | 26 | |
25 | 27 | |
| 28 | + |
| 29 | + |
| 30 | + |
| 31 | + |
| 32 | + |
| 33 | + |
| 34 | + |
| 35 | + |
| 36 | + |
| 37 | + |
| 38 | + |
| 39 | + |
| 40 | + |
| 41 | + |
| 42 | + |
| 43 | + |
| 44 | + |
| 45 | + |
| 46 | + |
| 47 | + |
| 48 | + |
| 49 | + |
| 50 | + |
| 51 | + |
| 52 | + |
| 53 | + |
| 54 | + |
| 55 | + |
26 | 56 | |
27 | 57 | |
28 | 58 | |
| ||
150 | 180 | |
151 | 181 | |
152 | 182 | |
153 | | - |
| 183 | + |
| 184 | + |
154 | 185 | |
Collapse file
lib/internal/main/check_syntax.js
Copy file name to clipboardExpand all lines: lib/internal/main/check_syntax.js+4-2Lines changed: 4 additions & 2 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
22 | 22 | |
23 | 23 | |
24 | 24 | |
| 25 | + |
| 26 | + |
| 27 | + |
| 28 | + |
25 | 29 | |
26 | 30 | |
27 | 31 | |
28 | 32 | |
29 | 33 | |
30 | 34 | |
31 | | - |
32 | | - |
33 | 35 | |
34 | 36 | |
35 | 37 | |
|
Collapse file
lib/internal/main/worker_thread.js
Copy file name to clipboardExpand all lines: lib/internal/main/worker_thread.js+4-1Lines changed: 4 additions & 1 deletion
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
6 | 6 | |
7 | 7 | |
8 | 8 | |
9 | | - |
| 9 | + |
| 10 | + |
10 | 11 | |
11 | 12 | |
12 | 13 | |
| ||
72 | 73 | |
73 | 74 | |
74 | 75 | |
| 76 | + |
| 77 | + |
75 | 78 | |
76 | 79 | |
77 | 80 | |
|
0 commit comments